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

Dernier appel pour la v22 #4173

Closed
vhf opened this issue Feb 4, 2017 · 16 comments
Closed

Dernier appel pour la v22 #4173

vhf opened this issue Feb 4, 2017 · 16 comments
Labels
Feedback Ticket ou PR en attente de retours

Comments

@vhf
Copy link
Contributor

vhf commented Feb 4, 2017

cc/ @zestedesavoir/les-bourgeons @zestedesavoir/les-jeunes-pousses

J'aimerais lancer la release v22 dans la semaine. Parmi les PRs ouvertes (à part ES, pièce de résistance de cette release), y a-t-il des choses qu'on pourrait encore y inclure ?

Merci pour tout votre travail !

@vhf vhf added the Feedback Ticket ou PR en attente de retours label Feb 4, 2017
@gllmc
Copy link
Member

gllmc commented Feb 4, 2017

Salut :)

En ce qui me concerne, il y a #4172 qui pourrait être utile pour optimiser la désinscription, mais rien qui ne relève de l'expérience utilisateur.

#4142 est aussi mergeable si quelqu'un fait la QA, mais ça ne concerne que les admins.

En tout cas, ça fera une belle release puisqu'il y a pas mal de petits changement assez sympas qui ont été codés. Pour la ZEP-13, ça ira dans la v23 du coup ?

@DevHugo
Copy link
Contributor

DevHugo commented Feb 4, 2017

Ouai, pour la ZEP, j'aurais jamais le temps de tout finir pour mercredi.

@artragis
Copy link
Member

artragis commented Feb 4, 2017 via email

@vhf
Copy link
Contributor Author

vhf commented Feb 4, 2017

Ok. Alors je résume mon plan:

On tente de lancer la release mercredi au plus tard. Reste à y intégrer :

Lors de la mise en beta de cette release, on active les pings et l'interface d'artragis. Puis une période de beta idéalement courte genre 1 semaine si tout va bien.

@tleb
Copy link
Contributor

tleb commented Feb 4, 2017

Je n'ai pas envoyé de patch pour le bug d'alignement vertical du bouton Suivre/Ne pas suivre, pardon. Je fais ça ce soir tard ou demain matin tôt.

@DevHugo
Copy link
Contributor

DevHugo commented Feb 4, 2017

Tu as jusqu'à mercredi soir, sinon ça passe dans la release d'aprés, y'a pas de souci.

@tleb
Copy link
Contributor

tleb commented Feb 5, 2017

#4082 a été résolu ?

@Anto59290
Copy link
Contributor

Si #4164 peut passer ça améliorerait un peu le menu. Sinon je pense que la PR sur l'interface de merge peut être QA, à voir si on à le temps de l'intégrer.

@vhf
Copy link
Contributor Author

vhf commented Feb 9, 2017

Pas pu m'occuper de la release. Je m'y colle ce week-end du coup, dès mon retour. Sauf s'il manque encore des trucs qu'il faudrait faire passer avec, auquel cas on peut encore un peu repousser.

@vhf
Copy link
Contributor Author

vhf commented Feb 10, 2017

Ok je suis de retour. @artragis on peut partir sans toi ou t'as une forte préférence pour retarder la release encore quelques jours ?

Ping @pierre-24 si t'as un avis ou si j'ai manqué un truc. J'ai rapidement fait un topo en passant sur mes notifs, mais j'ai peut-être raté un truc pendant mon absence.

@artragis
Copy link
Member

artragis commented Feb 10, 2017 via email

@vhf
Copy link
Contributor Author

vhf commented Feb 11, 2017

C'est fait.

@vhf vhf closed this as completed Feb 11, 2017
@pierre-24
Copy link
Member

pierre-24 commented Feb 11, 2017

Au vu de la page de recherche, je pense qu'il y a eu un truc avec la génération du CSS : sur le dernier tests en date de ES, la clem' de noël était présente sur la recherche (en plus du fait qu'il n'y a pas la marge entre le paginateur et la zone bleue, alors qu'elle y était). Me trompe-je ?

@vhf
Copy link
Contributor Author

vhf commented Feb 11, 2017

Bien vu. Aucune idée pour le CSS. Comme d'hab j'ai pas touché le front, il est buildé par Travis.

@Situphen
Copy link
Member

Alors je ne sais pas pourquoi ce n'était pas dans le PR sur le recherche, mais les correctifs pour le front sont dans #4196 !

@pierre-24
Copy link
Member

Voilà qui n'est pas logique:

screenshot from 2017-02-11 16 31 41

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feedback Ticket ou PR en attente de retours
Projects
None yet
Development

No branches or pull requests

8 participants