-
Notifications
You must be signed in to change notification settings - Fork 900
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
Db migration 2 #4252
Db migration 2 #4252
Conversation
3cf9292
to
8ae229e
Compare
8ae229e
to
51b0ecf
Compare
538408e
to
9d7c759
Compare
e912e75
to
662e486
Compare
d96a81d
to
0c322a1
Compare
f7b3e73
to
ee9e109
Compare
908062c
to
d88b0b3
Compare
d88b0b3
to
8459e95
Compare
Verified the following on macOS using build from: https://bravesoftware.slack.com/archives/CMH8DU4TF/p1582876908014700 Clean profile (anon wallet):
Clean profile connected, non-KYC'd user wallet
Clean profile KYC'd user wallet
Upgrade case 1: Install 1.0. Set up data (Ads panel, AC list, one time and recurring tips, wallet balance, wallet summary items, etc). Update to PR builder version.
Upgrade case 2: Install current Release version (1.4.95 at this moment). Set up data (Ads panel, AC list, one time and recurring tips, wallet balance, wallet summary items, etc). Update to PR builder version.
Upgrade Case 3: Install current Release version (1.4.95 at this moment). Disable AC via toggle.
Upgrade Case 4: Install current Release version (1.4.95 at this moment). Disable Ads via toggle.
Upgrade Case 5: Install current Release version (1.4.95 at this moment). Disable Rewards via toggle.
Encountered/Logged the following issues not mentioned above (but are not specific to this PR):
Clean profile connected, non-KYC'd user wallet
Clean profile KYC'd user wallet
|
🙏 |
Resolves brave/brave-browser#6363
Resolves brave/brave-browser#7217
Submitter Checklist:
npm run lint
)git rebase master
(if needed).git rebase -i
to squash commits (if needed).Test Plan:
Reviewer Checklist:
After-merge Checklist:
changes has landed on.