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

Les noms de groupes peuvent faire jusqu’à 100 caractères #981

Merged
merged 4 commits into from Mar 25, 2021

Conversation

niladic
Copy link
Collaborator

@niladic niladic commented Mar 24, 2021

No description provided.

@niladic niladic requested a review from jdauphant March 24, 2021 14:30
@jdauphant jdauphant temporarily deployed to aplus-demo-task-more-ch-jzcofz March 24, 2021 14:30 Inactive
@jdauphant jdauphant temporarily deployed to aplus-demo-task-more-ch-jzcofz March 24, 2021 14:59 Inactive
@@ -0,0 +1,5 @@
# --- !Ups
ALTER TABLE "user_group" ALTER COLUMN "name" SET DATA TYPE character varying(250);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ça passe bien visuellement dans l'interface de demande ?

Copy link
Member

@jdauphant jdauphant left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pas de soucis sur l'implémentation technique.

@DanielBMY @niladic La limite était calé pour l'affichage, il y a une description de groupe pour compenser.
Je recommande de vérifier 'affichage avec 250 caractères à la création de la demande (ça peut se faire avec le mode inspection de chrome).
Je vous laissse choisir ce qu'il y a de mieux avec ces nouveaux éléments.
image

@niladic
Copy link
Collaborator Author

niladic commented Mar 25, 2021

@jdauphant mon idée est que maintenant qu'on a fait ça : #937 , la limite de caractères pour le nom de groupe apporte moins de bénéfice, et qu'il est préférable de "responsabiliser" plutôt qu'interdire. Comme @DanielBMY est plutôt de ton avis, on a finalement décidé de passer à 100.

@jdauphant
Copy link
Member

ça passe bien à 138 caractères en tout cas, mieux que ce que je pensais

@jdauphant
Copy link
Member

c'est top le #937 , les gens peuvent modifier leurs groupes ?

limite à 100 chars
@jdauphant jdauphant temporarily deployed to aplus-demo-task-more-ch-jzcofz March 25, 2021 11:06 Inactive
limite à 100 chars
@jdauphant jdauphant temporarily deployed to aplus-demo-task-more-ch-jzcofz March 25, 2021 11:07 Inactive
@niladic
Copy link
Collaborator Author

niladic commented Mar 25, 2021

On n'a pas fait de modification de permissions si je me rappelle bien, donc le formulaire de nom de groupe n'est visible que par les responsables. Il y a un autre formulaire "mes groupes" qui est éditable par tout le monde par contre. Je referai une passe sur toutes les permissions avant l'audit.

@niladic niladic changed the title Les noms de groupes peuvent faire jusqu’à 250 caractères Les noms de groupes peuvent faire jusqu’à 100 caractères Mar 25, 2021
@niladic niladic merged commit b85b355 into master Mar 25, 2021
@niladic niladic deleted the task/more-chars branch March 25, 2021 14:36
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.

None yet

2 participants