-
Notifications
You must be signed in to change notification settings - Fork 873
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
Non-referral downloads should use generic refcode #1982
Conversation
@bbondy Two things to note here:
My reading of brave/brave-browser#2547 is that this is the expected/intended behavior, but if not please let me know! |
Yes reset after 90 days is expect. Thanks so much! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving but wait for security sign off.
if i'm not mistaken, this should be blocked on brave/vault-updater#90 and also https://github.com/brave/internal/issues/542#issuecomment-473574540, the latter of which i am working on |
@diracdeltas said she will take care of reviewing this. |
looks good to go as soon as brave/vault-updater#90 is closed |
a530c4c
to
406b822
Compare
Fixes brave/brave-browser#2547
Submitter Checklist:
npm test brave_unit_tests && npm test brave_browser_tests
) onnpm run lint
)git rebase master
(if needed).git rebase -i
to squash commits (if needed).Test Plan:
Download a non-referral stub installer (i.e., no referral code embedded in its filename)
Delete existing profile
Run the installer
After the installer completes, launch Brave and wait ~10 seconds
Confirm that
Local State
file (e.g.,/c/Users/emerick/AppData/Local/BraveSoftware/Brave-Browser-Development/User Data/Local State
) contains"promo_code":"BRV001"
entryConfirm with server team that daily ping for this run looked similar to this (in particular,
ref
isBRV001
):https://laptop-updates.brave.com/1/usage/brave-core?platform=winx64-bc&channel=release&version=0.64.4&daily=true&weekly=true&monthly=true&first=true&w oi=2019-03-11&ref=BRV001
Should also confirm that regular referral downloads continue to work as normal
Reviewer Checklist: