-
Notifications
You must be signed in to change notification settings - Fork 19
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Extend AccuradioDetector to bigcontentview #93
Comments
|
thank you for looking into this. :) |
Fixed in #94 |
much appreciated, will grab the .apk ASAP. |
Can you post a screenshot showing what is hard to read? |
And ideally a screenshot of the notification when the ad occurs (and not being detected). Will look again at the tracing dump file. It's a bit hard to troubleshoot as the issue seems device/ OS specific. I did not have this issue on my phone. |
The new generic text detector should now be able to detect any text in the notification, supposing the text is text and not an image. Enabling best effort and user defined text detector (with the ad string) does not fix the issue? |
The most recent tracing file does not contain an Ad with the text: "Music will resume shortly..". Did you record the old tracing file with the same version of Android 13? If so we do not need a new tracing file. Otherwise, could you create another tracing file and ensure that you can capture an ad?. many thanks :) |
moto g32 running Android 13. the log i originally sent in November had the device still on Android 12. i set battery optimisation to unrestricted, and always-on notifications to on, but the settings didn't apply until i restarted the device. after that, the notification appeared, as shown on screenshot, but ad was not muted. maybe this app's developers have managed something, to make the ad not register as one? text i can't read is shown here. sharing the log again, in case it contains something new after the reboot. |
i don't know if it will help, but for this log i restarted the station three times, so three "Music will resume shortly..." ads would play. |
AccuradioDetector only parses contentview, extend it to include (now deprecated) bigcontentview.
https://developer.android.com/reference/android/app/Notification#bigContentView
#83 @unseenlarks
The text was updated successfully, but these errors were encountered: