Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

431 Regex error message #432

Merged
merged 1 commit into from
Jul 14, 2023
Merged

Conversation

west270
Copy link
Contributor

@west270 west270 commented Jul 12, 2023

Closes #431
Added error message when parameter field value does not match regex pattern.

For testing the command echo_required_message_regex can be used to see that the error message appears. To get the error message to show, the form needs to be submitted with an error to trigger validation.

@west270 west270 requested a review from obr42 July 12, 2023 18:30
Copy link
Contributor

@obr42 obr42 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@obr42 obr42 merged commit fc62899 into beer-garden:main Jul 14, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Regex error message
2 participants