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

Titre de la page de modification du profil #4046

Merged
merged 2 commits into from
Dec 24, 2016
Merged

Titre de la page de modification du profil #4046

merged 2 commits into from
Dec 24, 2016

Conversation

gllmc
Copy link
Member

@gllmc gllmc commented Dec 9, 2016

Q R
Type de modification évolution

Cette pull request améliore la page d'édition d'un profil par un membre du staff. Elle modifie le titre (afin d'afficher le pseudo du membre modifié) et ajoute un message d'avertissement similaire à celui affiché lors de l'édition d'un message en tant que modérateur.

QA

  • Vérifier que la page de modification du profil d'un membre fonctionne correctement et que les changements évoqués ont bien été apportés ;
  • Vérifier que la page de modification du profil de l'utilisateur connecté fonctionne toujours.

@coveralls
Copy link

coveralls commented Dec 9, 2016

Coverage Status

Coverage increased (+0.0009%) to 87.463% when pulling bfb6324 on GCodeur:titre_modification_profil into cffb555 on zestedesavoir:dev.

@Anto59290
Copy link
Contributor

QA : NOK

En tirant ta branche et allant sur /membres/voir/admin/ avec le compte admin (admin/admin) ça provoque une erreur 500. Peux tu confirmer que tu as ça chez toi aussi ?

@gllmc
Copy link
Member Author

gllmc commented Dec 23, 2016

Ah bon ? Tout marche bien en ce qui concerne. C'est d'autant plus bizarre que je ne touche pas à cette vue. Les URL concernées par cette PR sont normalement /membres/parametres/mini_profil/{name}/ et /membres/parametres/profil/.

@Anto59290
Copy link
Contributor

QA : OK

Le problème venait de mon jeu de données, une fois nettoyé ça fonctionne comme attendu. Je suis désolé de mon erreur. Merci beaucoup ! :)

@gllmc
Copy link
Member Author

gllmc commented Dec 23, 2016

OK ! Et pas de problèmes :)

@vhf vhf added C-Front Concerne l'interface du site S-Évolution and removed QA svp labels Dec 24, 2016
@vhf
Copy link
Contributor

vhf commented Dec 24, 2016

Merci !

@vhf vhf merged commit a667134 into zestedesavoir:dev Dec 24, 2016
@vhf vhf added this to the Version de développement milestone Dec 25, 2016
@gllmc gllmc deleted the titre_modification_profil branch July 17, 2017 11:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Front Concerne l'interface du site
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants