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.

Sign up for GitHub

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

Jump to bottom

An exception occur if a phone or fax number is added in people form (create and edit) #318

Open
Yann-BUTSCHER-EIRL opened this issue Apr 6, 2023 · 1 comment
Open

An exception occur if a phone or fax number is added in people form (create and edit) #318

Yann-BUTSCHER-EIRL opened this issue Apr 6, 2023 · 1 comment
Assignees
@hundrex @LucileDT
Labels
Bug Something isn't working as it should be
Projects
Development

Comments

@Yann-BUTSCHER-EIRL
Copy link
Collaborator

Scenario

  • log in as admin
  • got to people create or edit page
  • enter a phone number
  • submit the form

Expected behavior

The form should be submitted and the data saved.

Current behavior

An error 500 occurs: The input given is not a phone number, only numbers allowed
@Yann-BUTSCHER-EIRL Yann-BUTSCHER-EIRL added the Bug Something isn't working as it should be label Apr 6, 2023
@Yann-BUTSCHER-EIRL
Copy link
Collaborator Author

After investigation this error is due to the input mask for phone numbers that add spaces between numbers.

The better approach would be to rework the people form entirely (see #319) but for now we'll just removed all spaces when saving the people info.

Yann-BUTSCHER-EIRL added a commit that referenced this issue Apr 6, 2023
@Yann-BUTSCHER-EIRL
Ref #318: remove space in phone numbers
bea2a31
@Yann-BUTSCHER-EIRL Yann-BUTSCHER-EIRL self-assigned this Apr 6, 2023
@Yann-BUTSCHER-EIRL Yann-BUTSCHER-EIRL added this to To do in Development via automation Apr 6, 2023
@Yann-BUTSCHER-EIRL Yann-BUTSCHER-EIRL moved this from To do to In progress in Development Apr 6, 2023
@Yann-BUTSCHER-EIRL Yann-BUTSCHER-EIRL moved this from In progress to Waiting tests in Development Apr 6, 2023
Yann-BUTSCHER-EIRL added a commit that referenced this issue May 2, 2023
@Yann-BUTSCHER-EIRL
Ref #318: remove space in phone numbers
55b2ee2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hundrex hundrex

@LucileDT LucileDT

Labels
Bug Something isn't working as it should be
Projects
No open projects
Development
  
Waiting tests
Milestone
No milestone
Development

No branches or pull requests
3 participants
@hundrex @LucileDT @Yann-BUTSCHER-EIRL