-
Notifications
You must be signed in to change notification settings - Fork 13
Terms Of Use Refactor CRASM-3304 #1282
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
Conversation
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.
I'd suggest moving the TermsOfUse.tsx file to the components/Dialog folder since it's not really a page anymore and removing the TermsOfUse folder under pages.
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.
A few things to address
…/XFD into TermsOfUse_Refactor-3304
@RKohler21 was this addressed? |
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.
Was this tested for 508 compliance
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.
Looks 508 compliant to me
4e9b897
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
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.
Second time around. Looks good

🗣 Description
This is a multi Purpose PR that closes both CRASM-3304 and CRASM-3290. For CRASM-3304, this PR contains a full refactor for our Terms Of Use page as well as the deletion of deprecated code from AuthForm and the removal of the /terms URL from our Route file. For CRASM-3290, this PR includes an fix that will resolve the issue of the First Login popup not appearing after accepting the terms of use but later appearing on the second login.
💭 Motivation and context
This PR includes updates/fixes for both PR's since they are heavily related to one another and will improve the flow of our code and UI experience going forward.
🧪 Testing
In order to test, the user will need to follow these steps when logging in for the first time on a new account.
📷 Screenshots (if appropriate)
Before refactor:


After refactor:


Warning Label Addition:

✅ Pre-approval checklist
bump_versionscript if this repository is versioned and the changes in this PR warrant a version bump.✅ Pre-merge checklist
✅ Post-merge checklist