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

Adds Rewards disclosure statement to onboarding process #69

Merged
merged 1 commit into from Sep 10, 2019
Merged

Conversation

@ryanml
Copy link
Member

ryanml commented Sep 6, 2019

Closes: brave/brave-browser#5901

This also fixed some issues with client state persistence

cc: @cg505

@ryanml ryanml force-pushed the fix-5901-2 branch from b8a578e to 5ca373d Sep 6, 2019
@ryanml ryanml requested a review from bbondy Sep 6, 2019
@ryanml ryanml self-assigned this Sep 6, 2019
@ryanml ryanml force-pushed the fix-5901-2 branch 3 times, most recently from 13dca6d to ec0c0d4 Sep 6, 2019
Copy link
Contributor

cg505 left a comment

Left a few comments, not 100% sure they are valid given I'm a bit little rusty with the codebase. Generally looks solid to me though

brave/ui/index.js Outdated Show resolved Hide resolved
brave/ui/app/ducks/metamask/metamask.js Outdated Show resolved Hide resolved
brave/ui/app/store/actions.js Outdated Show resolved Hide resolved
@ryanml ryanml force-pushed the fix-5901-2 branch from ec0c0d4 to a5b621d Sep 10, 2019
Fixes issues with state persistence
@ryanml ryanml force-pushed the fix-5901-2 branch from a5b621d to f9b7bf6 Sep 10, 2019
@ryanml ryanml requested a review from cg505 Sep 10, 2019
@cg505
cg505 approved these changes Sep 10, 2019
Copy link
Contributor

cg505 left a comment

nice!

@ryanml ryanml merged commit 48e0636 into master Sep 10, 2019
1 check passed
1 check passed
Travis CI - Pull Request Build Passed
Details
@ryanml ryanml deleted the fix-5901-2 branch Sep 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

2 participants
You can’t perform that action at this time.