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

[Android] Super Referrer - QR Code #10079

Closed
anthonypkeane opened this issue Jun 4, 2020 · 1 comment · Fixed by brave/brave-core#5741
Closed

[Android] Super Referrer - QR Code #10079

anthonypkeane opened this issue Jun 4, 2020 · 1 comment · Fixed by brave/brave-core#5741

Comments

@anthonypkeane
Copy link

Description

Super Referrer (SR) uses an enhanced referrer tracking system. On iOS, we do this by directing users to a landing page and copying a referral code.

The SR link is slightly different to the normal referrer link.

The QR code on the SR custom home page uses the normal referrer link, we need to change this so that URL link for the QR code can be entered in the .json file.

Link: https://bravesoftware.slack.com/archives/CT7JC4KBM/p1591227697202100

iOS issue: brave/brave-ios#2599

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Jun 17, 2020

Verification passed on Samsung Galaxy J3 version 5.1.1 running BraveModernarm 1.10.92.apk

  • Verified the test plan from Update QR code url brave-core#5741
  • Verified that the URL https://brave.com/r/OXB363 is sent to the user when code is shared via QR code menu from SR image
  • Verified that click on https://brave.com/r/OXB363 redirected to SR page

Verification passed on BraveModernarm v1.10.92 on Samsung Galaxy Tab S5e (Android 9.0)

  • Verified the test plan from Update QR code url brave-core#5741
  • Verified that the URL https://brave.com/r/OXB363 is sent to the user when code is shared via QR code menu from SR image
  • Verified that click on https://brave.com/r/OXB363 redirected to SR page

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants