-
Notifications
You must be signed in to change notification settings - Fork 2
Adding default password criteria to sample config #95
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
base: master
Are you sure you want to change the base?
Conversation
@minhducsun2002, this can be quite messy, you know |
Hardcoding checks in the web app is not a good way to implement criteria for strings. The more preferred method is a regular expression. |
But hey, I can't find your implementation. There's only sample change |
I see that you seem to be serving If you mean to add checks for this feature or Enquire entries to change that, sorry, working on that. |
That's not how the config system work. Currently, it parses config while preserving as much as possible. That means it only read white-listed properties. See in src/util/config/contestConfig.js for more details. |
Should work properly now. |
Hmm, anyone? |
@MegumiTadokoro Feel free to contribute, at least after rebasing or resolving conflict |
This enables setting custom rules for passwords.
However, whether this option is mandatory in the configuration is up to upstream maintainers.