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 align on Suivre button #4174

Closed
wants to merge 1 commit into from
Closed

Fix align on Suivre button #4174

wants to merge 1 commit into from

Conversation

tleb
Copy link
Contributor

@tleb tleb commented Feb 5, 2017

Q R
Type de modification correction de bug
Ticket(s) (issue(s)) concerné(s) cf Forum

QA

Vérifier l'aspect du bouton Suivre sur les différentes pages où il apparaît : page d'un forum, page d'un sujet, page d'un user, page d'un article, page d'un tuto et page d'un billet (j'ai rien oublié ?).

@DevHugo DevHugo self-assigned this Feb 5, 2017
@DevHugo DevHugo added C-Front Concerne l'interface du site QA svp S-BUG Corrige un problème labels Feb 5, 2017
@coveralls
Copy link

coveralls commented Feb 5, 2017

Coverage Status

Coverage remained the same at 88.363% when pulling 99c1d9b on tleb:dev into 807fa2d on zestedesavoir:dev.

@DevHugo
Copy link
Contributor

DevHugo commented Feb 5, 2017

Salut,

Tu as oublié de créer une branche. Du coup, tu as fait tes modifs direct sur la branche dev et je ne peux pas merger, sans tout casser ^^ !

C'est un peu détaillé dans la documentation, dans la section "Créer une branche pour faire vos modifications".

Si tu as un souci pour appliquer le workflow, n'hésite pas a me faire signe et je tenterait d'expliciter.

En attendant, j'ai créé une deuxième PR avec les modifications sur une branche.

QA: Ok

@tleb
Copy link
Contributor Author

tleb commented Feb 5, 2017

Oups, j'ai fais ça rapidement ce matin avant de partir, j'ai même pas pris le temps de créer une nouvelle branche, sorry.

@DevHugo DevHugo closed this Feb 5, 2017
@DevHugo
Copy link
Contributor

DevHugo commented Feb 5, 2017

Pas de souci :p

@DevHugo
Copy link
Contributor

DevHugo commented Feb 5, 2017

Merci pour la PR et à une prochaine.

@gllmc gllmc removed the QA svp label Apr 3, 2017
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 S-BUG Corrige un problème
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants