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

INT B-22668 #14950

Open
wants to merge 7 commits into
base: integrationTesting
Choose a base branch
from
Open

INT B-22668 #14950

wants to merge 7 commits into from

Conversation

danieljordan-caci
Copy link
Contributor

Agility ticket

Feature Flag PR

Summary

We are rolling out customer self registration where we will have a little more control over form validations than we do in Okta. This PR specifically deals with the UI form, validations, and setting up feature flag work. No backend work involving the form submission or account creation is done in this BL.

This PR does the following:

  • adds a customer_registration feature flag
  • adds a FF endpoint that does not require authentication, this was needed since we are rendering the sign up button outside of auth
  • adds a few components used in the registration form: StyledLine.jsx, SignUp.jsx, ValidCACModal.jsx, RegistrationConfirmationModal.jsx
  • updates TextField.jsx to not allow pasting when prop disablePaste is provided
  • tests

How to test

  1. Turn on the FEATURE_FLAG_CUSTOMER_REGISTRATION in your envrc
  2. Fire up your server & client
  3. You should see a Sign up button on the main page of the customer app
  4. Click it
  5. Accept terms
  6. Should now be directed to the sign up page where a CAC modal will immediately pop up
  7. Click no
  8. Should be kicked back to the home page with a red banner letting you know you need a CAC
  9. Click Sign up again
  10. Accept terms
  11. Click Yes on the CAC modal
  12. Should now see the registration form
  13. Ensure all validations are in place and you can't submit the form without matching DODIDs/EMPLIDs/emails
  14. Confirm you can't copy/paste in the confirm inputs
  15. Submit the form, should see some loading spinners
  16. You'll be redirected to Okta
  17. NOTE: this is just a placeholder for now, but once the next BL is complete then we will complete the backend work of creating MM & Okta accounts - this BL stops here

Screenshots

Desktop
https://github.com/user-attachments/assets/fcde2dee-026a-47e5-89a1-70a87513a69a

Mobile
https://github.com/user-attachments/assets/6f031975-bd9b-4e2a-a621-389b006890a7

Screenshot 2025-03-03 at 5 40 31 PM Screenshot 2025-03-03 at 5 40 37 PM Screenshot 2025-03-03 at 5 40 46 PM Screenshot 2025-03-03 at 5 40 52 PM Screenshot 2025-03-03 at 6 08 36 PM

Screenshot 2025-03-04 at 2 22 16 PM
Screenshot 2025-03-04 at 2 22 41 PM

@danieljordan-caci danieljordan-caci added Mountain Movers Movin' Mountains 1 Sprint at a time INTEGRATION Slated for Integration Testing labels Mar 4, 2025
@danieljordan-caci danieljordan-caci requested review from loganwc, antgmann and a team March 4, 2025 20:32
@danieljordan-caci danieljordan-caci self-assigned this Mar 4, 2025
@danieljordan-caci danieljordan-caci requested a review from a team as a code owner March 4, 2025 20:32
@danieljordan-caci danieljordan-caci requested a review from a team as a code owner March 4, 2025 20:32
@danieljordan-caci danieljordan-caci requested a review from a team as a code owner March 4, 2025 20:32
@danieljordan-caci danieljordan-caci mentioned this pull request Mar 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
INTEGRATION Slated for Integration Testing Mountain Movers Movin' Mountains 1 Sprint at a time
Development

Successfully merging this pull request may close these issues.

1 participant