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

Fix #3395 : Corrige le message de la page d'inscription #3425

Closed
wants to merge 7 commits into from
Closed

Fix #3395 : Corrige le message de la page d'inscription #3425

wants to merge 7 commits into from

Conversation

MatthieuLepers
Copy link

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

notes
Le "Membres" dans le fils d'ariane est toujours présent, @Wyzko est en train de bosser dessus.

<h2>Inscription</h2>
<p class="alert-box warning">
<strong>
{% trans "Vous êtes déjà inscrit" %}.
Copy link
Member

Choose a reason for hiding this comment

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

Peux-tu mettre le "." dans la traduction, notamment pour les langes plus exotiques, si un jour on en a?

@MatthieuLepers
Copy link
Author

C'est fait.

<p>
{% trans "Vous êtes déjà connecté" %}.
</p>
<div>
Copy link
Member

Choose a reason for hiding this comment

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

Dans tout le projet on utilise 4 espaces au lieu d'une tabulation ! (Ici il en faut 12 pour que ça dépasse le {% else %})

@Situphen
Copy link
Member

Situphen commented Mar 5, 2016

Est-ce que tu peux mettre un titre plus explicite que "Fix #3395" (par exemple, "Corrige le message de la page d'inscription") ?
Peux-tu squasher tes deux premiers commits ?
:)

@gustavi
Copy link
Contributor

gustavi commented Mar 6, 2016

Je rajouterai que des notes de QA sont les bienvenues.

@MatthieuLepers MatthieuLepers changed the title Fix #3395 Fix #3395 : Corrige le message de la page d'inscription Mar 7, 2016
@MatthieuLepers
Copy link
Author

@Situphen Je ne maîtrise pas encore le squash sur git, il faudra que je me renseigne ou que je demande à quelqu'un sachant le faire.

@artragis
Copy link
Member

artragis commented Mar 7, 2016

Si tu es présent sur IRC, j'ai deux secondes.

Le 7 mars 2016 à 17:13, Matthieu Lepers notifications@github.com a écrit :

@Situphen https://github.com/Situphen Je ne maîtrise pas encore le
squash sur git, il faudra que je me renseigne ou que je demande à quelqu'un
sachant le faire.


Reply to this email directly or view it on GitHub
#3425 (comment)
.

@SpaceFox
Copy link
Contributor

SpaceFox commented Mar 7, 2016

Il devient vraiment urgent qu'avec @Eskimon on écrive ce tuto sur Git. Un qui ne se contente pas d'expliquer la CLI.

@Situphen
Copy link
Member

Situphen commented Mar 7, 2016

On a déjà un petit tutoriel fait par @Eskimon mais c'est vrai qu'il en faudrait un complet.

@orandin
Copy link
Contributor

orandin commented Mar 7, 2016

@AireAyquaza : N'hésite pas à créer une nouvelle branche de test depuis ta branche Correction-3395 (git checkout -b nom-de-ta-branche sans le upstream/dev) pour t'entrainer à la réécriture d'historique de git. Si tu fais une mauvaise manip', ce n'est pas grave. Tu pourras détruire la branche de test et recommencer. :-)

Une fois que tu auras cerné le fonctionnement, squash ta branche Correction-3395. Si tu n'y arrives pas, je t'expliquerai et on le fera ensemble. ;-)

@coveralls
Copy link

Coverage Status

Coverage increased (+0.03%) to 87.845% when pulling bcffb28 on AireAyquaza:Correction-3395 into 9a844fb on zestedesavoir:dev.

@SpaceFox
Copy link
Contributor

Hmm, j'ai des merge bizarres dans la liste des commits, et Travis n'est pas content. Puis-je avoir un statut sur cette PR ?

@artragis
Copy link
Member

Travis semble heureux.

@GerardPaligot
Copy link
Member

L'historique reste bizarre.

@Emeric54
Copy link
Contributor

La nouvelle PR de @Wyzko reprend l'issue, on peut donc fermer ici.

@gustavi gustavi closed this Mar 27, 2016
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

10 participants