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

Adding risk disclosure statement to password components #38

Merged
merged 1 commit into from Aug 23, 2019
Merged

Conversation

@ryanml
Copy link
Member

ryanml commented Aug 12, 2019

@ryanml ryanml requested review from tomlowenthal and bbondy Aug 12, 2019
@ryanml ryanml self-assigned this Aug 12, 2019
@ryanml
Copy link
Member Author

ryanml commented Aug 12, 2019

Note: https://brave.com/cryptocurrency-risk currently leads to a 404, but my assumption is that the page will be live before we go to nightly cc: @tomlowenthal

@ryanml ryanml force-pushed the fix-5415 branch from 1179520 to ed932f0 Aug 12, 2019
@tomlowenthal
Copy link
Member

tomlowenthal commented Aug 12, 2019

@ryanml Yes, I need to post that page. Content is already written, but I need to get it on the site.

@cg505
cg505 approved these changes Aug 12, 2019
Copy link
Contributor

cg505 left a comment

👍👍👍

@bbondy bbondy force-pushed the master branch 4 times, most recently from 0e7f3a6 to ae3de68 Aug 13, 2019
@ryanml ryanml force-pushed the fix-5415 branch from ed932f0 to a58992f Aug 13, 2019
@bbondy
bbondy approved these changes Aug 13, 2019
@ryanml ryanml force-pushed the fix-5415 branch from a58992f to 6f28db1 Aug 23, 2019
@ryanml ryanml merged commit 6034634 into master Aug 23, 2019
1 check failed
1 check failed
Travis CI - Pull Request Build Failed
Details
@ryanml ryanml deleted the fix-5415 branch Aug 23, 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.

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