Skip to content
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

Tapping Ad notification doesn't launch beta app #7588

Open
srirambv opened this issue Jan 3, 2020 · 15 comments
Open

Tapping Ad notification doesn't launch beta app #7588

srirambv opened this issue Jan 3, 2020 · 15 comments

Comments

@srirambv
Copy link
Collaborator

@srirambv srirambv commented Jan 3, 2020

Description

Tapping Ad notification doesn't launch beta app

Steps to Reproduce

  1. Install Android-core beta and enable rewards
  2. Wait for an acutal ad notification to show up (not the sample ad notification)
  3. Close Brave-beta from recent apps
  4. Open notification center and tap on the ad notification, Doesn't launch Brave-beta

Actual result:

Tapping Ad notification doesn't launch beta app

Expected result:

Should launch Brave-Beta if app is not running in memory

Reproduces how often:

Easy

Brave version (brave://version info)

Android-Core beta

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields?
  • Does the issue resolve itself when disabling Brave Rewards?
  • Is the issue reproducible on the latest version of Chrome?

Miscellaneous Information:

cc: @gdregalo @samartnik

@srirambv srirambv added this to To Do in Android General via automation Jan 3, 2020
@srirambv srirambv added this to the Android-Beta milestone Jan 7, 2020
@emerick emerick self-assigned this Jan 8, 2020
@srirambv srirambv added the priority/P2 label Jan 8, 2020
@srirambv srirambv removed this from the Android-Beta milestone Jan 8, 2020
@emerick
Copy link
Contributor

@emerick emerick commented Jan 22, 2020

@srirambv Are you also seeing this behavior in the nightlies? I'm having trouble reproducing it, not sure if I'm missing a step...? When I exit the browser and tap on an ad, the browser relaunches with the ad (although there is a 2-3 second pause between tapping the ad and the browser launching).

@srirambv
Copy link
Collaborator Author

@srirambv srirambv commented Jan 22, 2020

Haven't checked it on nightly will run through it. Will check it on nightly and update here

@emerick
Copy link
Contributor

@emerick emerick commented Jan 22, 2020

@srirambv Sounds good, thank you!

@srirambv
Copy link
Collaborator Author

@srirambv srirambv commented Jan 22, 2020

@emerick I can reproduce this on Nightly CI build as well. Both with sample and actual ad notification as well

@emerick
Copy link
Contributor

@emerick emerick commented Jan 22, 2020

@srirambv OK, thanks for confirming - trying the nightly now, just waiting for an ad to show up...

@emerick
Copy link
Contributor

@emerick emerick commented Feb 12, 2020

@srirambv Finally got back to this and it's not happening for me with the latest nightly v1.6.21. If you can still repro, can you make a video showing how it happens from beginning to end? If not possible, maybe we can jump on a video call to do the same?

@srirambv
Copy link
Collaborator Author

@srirambv srirambv commented Feb 25, 2020

@emerick shared logs and video of issue reproducing on slack

@emerick
Copy link
Contributor

@emerick emerick commented Feb 25, 2020

@srirambv I tried with a locally built BraveMonoarm64.apk running on Android 10 and with the previous beta version (1.5.97). In both scenarios, clicking notifications when Brave wasn't running correctly launched Brave. I think there must be some other difference we're missing here. Is the behavior intermittent for you? Any other differences you can think of?

@SergeyZhukovsky
Copy link
Member

@SergeyZhukovsky SergeyZhukovsky commented Feb 26, 2020

I use Beta as my daily driver on my main device and ads clicks work as expected for me. @srirambv do you see it on some particular device or you can replicate on all Android devices you have?

@srirambv
Copy link
Collaborator Author

@srirambv srirambv commented Feb 28, 2020

@SergeyZhukovsky I see the issue on my OnePlus 6T with Android 10. I've shared the recording of the issue with @emerick and the logs.

I don't seem to see any other difference apart from installing the apk. I haven't made any changes to notifications as well at device level

@srirambv
Copy link
Collaborator Author

@srirambv srirambv commented Feb 28, 2020

I can see this issue happen on current GPS build as well on my device

@GeetaSarvadnya
Copy link
Collaborator

@GeetaSarvadnya GeetaSarvadnya commented Feb 28, 2020

Could not reproduce the issue in Samsung Galaxy J3 version 5.1.1 running Android stable 1.5.100 and GPS version 1.5.6

@emerick
Copy link
Contributor

@emerick emerick commented Mar 3, 2020

I wasn't able to repro with a OnePlus 6T device, unfortunately. I tried with Android 9 (which it shipped with) and Android 10 (after installing the updates). I tried debug and release builds and regular ARM and Monoarm64 builds with no luck.

@emerick
Copy link
Contributor

@emerick emerick commented Mar 5, 2020

Spoke with @SergeyZhukovsky and we're going to move this out of the upcoming 3/17 release as it also happens in browser-android-tabs per https://bravesoftware.slack.com/archives/GUA24E92P/p1582872239001400 and #7588 (comment).

@kjozwiak
Copy link
Member

@kjozwiak kjozwiak commented Mar 12, 2020

I can't seem to reproduce this on my Samsung Galaxy S10+ using 1.5.109 CR: 80.0.3987.132 either. Went through the following:

  • had three different ad notifications listed under the notifications tray from Beta
  • closed 1.5.109, tapped on the first notification and 1.5.109 oepened
  • repeated the above three times and 1.5.109 opened every single time

@srirambv do you see it on some particular device or you can replicate on all Android devices you have?

@srirambv can you reproduce this on other devices? Or just on the OnePlus 6T device?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Android General
  
Backlog
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
6 participants
You can’t perform that action at this time.