-
Notifications
You must be signed in to change notification settings - Fork 1
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
Summary character length should be enforced #2
Comments
One difficulty of this is currently fields used by the form (and constraints such as whether they are required) are stored on the backend in a hardcoded structure. We should make this configurable via the UI. Either way, we will need to figure out how to encode these constraints into the structure. One idea is:
In the above case we add (Note: in the structure the |
Since version cfp-app 1.1.4 we warn about exceeding the Summary field length, but this is a one-off hardcoded check (and for now we do not enforce the length limitation on either the client or serverside). |
Currently the character length limit of the CFP summary is stated, but not enforced, and it should be.
It should probably also be configurable.
The text was updated successfully, but these errors were encountered: