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

Add client-side validation (min/max for string and number options) #530

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

hyper-neutrino
Copy link
Contributor

Summary of Changes

  • added client-side validation options for string and number options (min/max length or value)

Motivation and Explanation

  • certain command options such as ban purge duration (not applicable after improve ban command input #528 in this case) allow invalid inputs and only detect the error afterwards
  • this change adds client-side validation to improve the user experience and understanding

Demonstration of Changes

image

Further Information and Comments

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.

1 participant