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

Suppression d'un lien en double dans un template #2416

Merged
merged 1 commit into from Mar 11, 2015

Conversation

@DevHugo
Copy link
Contributor

commented Mar 11, 2015

Q R
Correction de bugs ? oui
Nouvelle Fonctionnalité ? non
Tickets (issues) concernés

Comment reproduire ?

  • Création d'un compte
  • Valider le compte
  • Vérifier que dans la page il n'y a pas deux fois le lien comme montré dans la capture

bug2liens

@landscape-bot

This comment has been minimized.

Copy link

commented Mar 11, 2015

Code Health
Code quality remained the same when pulling e152b07 on DevHugo:suppression_lien_activation into e1f4d75 on zestedesavoir:dev.

@GerardPaligot

This comment has been minimized.

Copy link
Member

commented Mar 11, 2015

Ce bug était inconnu au bataillon. On peut le reproduire sur quelle version ? Sur la branche prod, sur la branche dev ? La manipulation pour le reproduire est la manipulation exacte que tu spécifies dans ton premier message ?

@firm1

This comment has been minimized.

Copy link
Contributor

commented Mar 11, 2015

Je pensais que c'était une feature ce truc.

@DevHugo

This comment has been minimized.

Copy link
Contributor Author

commented Mar 11, 2015

Ce bug était inconnu au bataillon. On peut le reproduire sur quelle version ?

Sur dev et prod, (en tout cas sur la branche nommé "prod").

La manipulation pour le reproduire est la manipulation exacte que tu spécifies dans ton premier message ?

Ouai, juste créé un compte et le valider, le problème est sur le templates de validation du token.

C'est juste un lien "Mot de passe oublié ?" en double dans un templates, rien de bien grave. Un truc mieux serait de connecter directement l'utilisateur.

@pierre-24

This comment has been minimized.

Copy link
Member

commented Mar 11, 2015

Sinon, rapport de QA: OK. À merger, sauf si ... Vous êtes pas d'accord sur le fait que c'est un bug o_O

@GerardPaligot

This comment has been minimized.

Copy link
Member

commented Mar 11, 2015

Non, non. C'est bien un bug. Je trouvais ça juste bizarre d'avoir une PR correction d'un bug inconnu. :)

Je merge du coup.

GerardPaligot added a commit that referenced this pull request Mar 11, 2015
Merge pull request #2416 from DevHugo/suppression_lien_activation
Suppression d'un lien en double dans un template

@GerardPaligot GerardPaligot merged commit 732bb29 into zestedesavoir:dev Mar 11, 2015

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@GerardPaligot GerardPaligot added this to the Version 1.7 milestone Mar 11, 2015

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