Skip to content

Conversation

Copy link

Copilot AI commented Oct 8, 2025

Updated the validation logic for the "add-value" field in the PopUp component to require exactly 10 characters instead of 5 characters for location numbers.

Changes

  • Modified the handleChange method in src/PopUp.js to update the validation rules for the add-value case
  • Changed the minimum length requirement from 5 to 10 characters
  • Changed the maximum length requirement from 5 to 10 characters
  • Updated error messages to reflect the new 10-character requirement

Details

The validation now enforces:

  • If the input length is less than 10 characters: displays "Location number must be 10 characters long!"
  • If the input length is greater than 10 characters: displays "Location number can not be more than 10 characters long!"
  • If the input is exactly 10 characters: no error is displayed

Note: The edit-value validation remains unchanged and still requires 5 characters as it was not part of the requested change.

Original prompt

Create a PR to change the add-value change to require at least 10 characters instead of 5


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

@JonCGit JonCGit marked this pull request as ready for review October 8, 2025 20:31
Copilot AI changed the title [WIP] Update add-value to require at least 10 characters Change add-value validation to require 10 characters instead of 5 Oct 8, 2025
Copilot AI requested a review from JonCGit October 8, 2025 20:32
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 this pull request may close these issues.

2 participants