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

Updates to phone number fields on Admin #2560

Closed
9 tasks
nickaddy opened this issue Sep 12, 2024 · 4 comments · Fixed by #2594
Closed
9 tasks

Updates to phone number fields on Admin #2560

nickaddy opened this issue Sep 12, 2024 · 4 comments · Fixed by #2594
Assignees

Comments

@nickaddy
Copy link
Contributor

Background

In this ticket, we have made some changes to fields on Apply; these need to be reflected in the Application view and edit on Admin.

Feature(s) Description

  • Existing phone number field no longer appears in Application view
  • New mobile phone number field appears in Application view
  • New mobile phone number field is editable on Application view

Definition of Done

  • User stories/acceptance criteria met
  • Internal reviews passed (feedback actioned)
  • User testing passed
  • Relevant technical documentation updated
  • User guidance updated
  • Deployed and merged without errors

User Testing Steps

Specify for users what is being tested (but not how to test it.)

Feature Champion

Nick

@nickaddy
Copy link
Contributor Author

@drieJAC
Copy link
Contributor

drieJAC commented Sep 16, 2024

@nickaddy dont we need to support both mobile phone number and phone number for new and legacy applications?
Also we will need to have validation for the new mobile phone number field

@nickaddy
Copy link
Contributor Author

@drieJAC I think so, yes, that's why I suggested just not displaying the old phone number field rather than deleting it. Admins will still see the number on Admin only not the candidates on Apply.

The validation for the new field is covered in the 2FA ticket.

@drieJAC drieJAC self-assigned this Oct 28, 2024
@drieJAC
Copy link
Contributor

drieJAC commented Oct 28, 2024

@nickaddy this also needs to be implemented in the candidates page in admin so I'm going to up this ticket estimate to a 5

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants