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

Respect du nombre de caractères maxi des champs rédactionnels #2901

Closed
numahell opened this issue Jan 18, 2022 · 2 comments · Fixed by #3052
Closed

Respect du nombre de caractères maxi des champs rédactionnels #2901

numahell opened this issue Jan 18, 2022 · 2 comments · Fixed by #3052
Assignees

Comments

@numahell
Copy link

Contrôler le respect du nombre de caractères (espaces compris) maximum sous les champs rédactionnels, et indiquer le nombre max.

CM : Activable ou non et nombre de caractères paramétrable.

@camillemonchicourt
Copy link
Member

Fait dans la 2.86.0 avec l'ajout d'un paramètre MAX_CHARACTERS et documenté : https://geotrek.readthedocs.io/en/2.86.0/install/advanced-configuration.html#max-characters-count

Celui-ci s'applique à tous les champs texte de tous les modules.
Cela peut être limitant car on peut vouloir limiter la longueur des descriptions des randos pour une cohérence éditoriale, mais sans aucun intérêt à appliquer les mêmes limites aux descriptions des chantiers, interventions, signalétique... où il n'y a pas d'enjeu éditorial.

Et même pour un même module, on peut vouloir appliquer une limite de 1000 caractères pour la description, mais de 200 caractères pour le chapeau.

A voir à terme si il faut pouvoir préciser ce paramètre par module, voire par champs. Un peu complexe, mais possiblement nécessaire pour rendre cette fonctionnalité pertinente et opérationnelle.

@camillemonchicourt
Copy link
Member

Ce fonctionnement a été revu et amélioré dans la version 2.102.2 pour être défini champs par champs avec le nouveau paramètre MAX_CHARACTERS_BY_FIELD qui remplace le paramètre MAX_CHARACTERS.

Voir #3844

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 a pull request may close this issue.

2 participants