-
Notifications
You must be signed in to change notification settings - Fork 70
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
SPIKE: CMS: Phone number validation and guidance #15691
Comments
per Michelle in Slack |
@BerniXiongA6 @EWashb FYI: Facilities cut this ticket today, not high priority on our end but documenting the conversation we had about opportunity here. Both CMS and Facilities teams are tagged, as we (Fac) could pursue in future via CMS Collab Cycle, or y'all might opt to look into it. |
Thank you @jilladams! This may be related to some things that @laflannery was doing regarding an audit, etc. May want to check in with her as well. |
From @laflannery audit: There is some existing validation on the Phone paragraph type fields.
|
@davidmpickett @jilladams @omahane @laflannery Supposedly this is blocking a ticket that is blocking VBA pilot rollout. Do we really not have an agreed path forward on phone numbers or can we close this and move on? |
This is an old ticket that can probably be retired. Laura's audit covered a lot of use cases. |
User Story or Problem Statement
As a CMS Editor, I want to clearly understand how phone numbers should be entered in the CMS, so that consistent information is provided to Veterans.
Description or Additional Context
Right now Drupal does not enforce any validation on Phone numbers. Phone numbers are entered as strings, and for Facility content types, Lighthouse receives / delivers phone numbers as strings.
Phone numbers have complex use cases, so we need to consider the use cases before implementing validation. A not-comprehensive list that can be expanded on:
*
optionsVA Design system phone number guidance
https://design.va.gov/content-style-guide/dates-and-numbers#phone-numbers
As of 10/16/23:
Affected content types / entities
Steps for Implementation
Acceptance Criteria
The text was updated successfully, but these errors were encountered: