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

Autocomplétion mp validation du formulaire #2355

Closed
poulp opened this issue Feb 18, 2015 · 12 comments

Comments

@poulp
Copy link
Contributor

commented Feb 18, 2015

Quand je veux écrire un mp et que j'ajoute un participant via l'auto-complétion, par exemple user, ça me donne user, ce qui invalide le formulaire. Il faudrait juste avoir user.

@GerardPaligot

This comment has been minimized.

Copy link
Member

commented Feb 18, 2015

Via la bêta ?

@poulp

This comment has been minimized.

Copy link
Contributor Author

commented Feb 18, 2015

Oui c'est présent sur la bêta

@SpaceFox SpaceFox added this to the "Futur proche" (v1.x) milestone Feb 18, 2015

@SpaceFox

This comment has been minimized.

Copy link
Member

commented Feb 18, 2015

Je ratisse large, il faudra dégager les labels non pertinents.

@GerardPaligot

This comment has been minimized.

Copy link
Member

commented Feb 18, 2015

C'est clairement du front. Le back, plus précisément l'API, ne fait que renvoyer un tableau avec la liste des utilisateurs. Rien de plus.

@Eskimon

This comment has been minimized.

Copy link
Member

commented Feb 19, 2015

A noter : le comportement de l'autocompletion est le même en prod (ajout de la virgule etou) mais le back se comporte bien, le message est bien envoyé.

@Eskimon

This comment has been minimized.

Copy link
Member

commented Feb 19, 2015

Un git blame nous indique que la derniere modification date d'il y a 14 jours par...@poulp !

https://github.com/zestedesavoir/zds-site/blame/dev/zds/mp/forms.py#L79

C'est ce commit qui semble embeter le formulaire : 8be73f7 dans cette PR : #2216

@GerardPaligot

This comment has been minimized.

Copy link
Member

commented Feb 19, 2015

Hum, sorry.

@Eskimon

This comment has been minimized.

Copy link
Member

commented Feb 19, 2015

git blame, la commande du petit délateur !

@poulp

This comment has been minimized.

Copy link
Contributor Author

commented Feb 19, 2015

Oui effectivement c'est moi qui ai mis en place la contrainte au niveau du formulaire en oubliant de changer le comportement du front

@Eskimon

This comment has been minimized.

Copy link
Member

commented Feb 19, 2015

(J'en profite pour m'excuser éventuellement pour mon comportement (histoire qu'on soit sur la même longueur d'onde), je ne jette pas la pierre, je cherche juste a trouver la solution :) . Que ce soit toi ou un autre qui a introduit le "problème" c'est pas bien grave ^^ . Le reste n'est qu'humour bien entendu).

Le fix sera a faire sur la branche de release.

@poulp

This comment has been minimized.

Copy link
Contributor Author

commented Feb 19, 2015

On est sur la même longueur d'onde, pas de soucis ;)

@SpaceFox SpaceFox modified the milestones: Version 1.6, "Futur proche" (v1.x) Feb 20, 2015

@SpaceFox

This comment has been minimized.

Copy link
Member

commented Feb 20, 2015

Corrigé.

@SpaceFox SpaceFox closed this Feb 20, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.