-
Notifications
You must be signed in to change notification settings - Fork 5
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
Added ToS accepting before log in (with help from Noel <3) #70
Conversation
…s a package to use the TermsOfService.md and added tos to the router inside index.ts.
# Conflicts: # src/router/index.ts
…er components unless the page we are on is the tos-page.
…dashboardlayout when tos is not accepted.
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.
LGTM!
Added ToS accepting using frontend-common functions, now requires this pull request to be merged first! (Should also be why the checks are failing) |
Oh and this also fixes the issue where if you reload, your balance doesn't update. The issue was |
Cannot be approved until frontend-common is updated.
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.
lgtm
🎉 This PR is included in version 1.0.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Fixes #60