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

Dynamic limit of event description field #935

Closed
ebartz opened this issue Jul 19, 2020 · 0 comments · Fixed by #939
Closed

Dynamic limit of event description field #935

ebartz opened this issue Jul 19, 2020 · 0 comments · Fixed by #939

Comments

@ebartz
Copy link
Contributor

ebartz commented Jul 19, 2020

Is your feature request related to a problem? Please describe.
It would be nice if the frontend takes te actual size of a database field into account.

For instance, the limit for the description of an event is currently hard coded to 2048, while the database field could be larger:

<textarea data-ng-model="obj.description[allLanguagesMapping[langValue].locale]" name="description-{{allLanguagesMapping[langValue].locale}}" id="description" required class="form-control" data-ng-minlength="10" data-ng-maxlength="2048"></textarea>

Describe the solution you'd like
The application could check on startup how big the database field actually is.

Better solution
Let the user increase the limit in the admin backend and actually perform the change on the database if needed.

@cbellone cbellone added this to the 2.0-M3 milestone Jul 21, 2020
cbellone added a commit to cbellone/alf.io that referenced this issue Aug 6, 2020
cbellone added a commit to cbellone/alf.io that referenced this issue Aug 18, 2020
cbellone added a commit to cbellone/alf.io that referenced this issue Aug 18, 2020
cbellone added a commit to cbellone/alf.io that referenced this issue Aug 18, 2020
syjer pushed a commit that referenced this issue Aug 18, 2020
* check if check-in is allowed before evaluating ticket status

* #935 - change columns type to TEXT

* #935 - dynamic limit for event description

* #935 - validate ticket category

* #935 - rename script
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants