Derniers messages sur Zeste de Savoirhttps://zestedesavoir.com/forums/2014-07-28T00:55:19+02:00Les derniers messages parus sur le forum de Zeste de Savoir.Certaines PR sont à valider par les admin sys., message #121232014-07-28T00:55:19+02:00Alex-D/@Alex-Dhttps://zestedesavoir.com/forums/sujet/831/certaines-pr-sont-a-valider-par-les-admin-sys/?page=1#p12123<figure><blockquote>
<p>Pour la première PR, j'ai fait certaine remarques à prendre en compte.
</p>
</blockquote>
<figcaption><p><a href="http://zestedesavoir.com/forums/sujet/831/certaines-pr-sont-a-valider-par-les-admin-sys/?page=1#p11435">firm1</a></p></figcaption></figure><p>Du coup j'imagine qu'il faut que tu mettes à jour celle-ci pour que firm1 la valide.</p>Certaines PR sont à valider par les admin sys., message #121052014-07-27T23:57:57+02:00SpaceFox/@SpaceFoxhttps://zestedesavoir.com/forums/sujet/831/certaines-pr-sont-a-valider-par-les-admin-sys/?page=1#p12105<p>Je ne peux pas valider la première, puisque c'est moi qui l'ai faite…</p>Certaines PR sont à valider par les admin sys., message #117012014-07-26T01:27:43+02:00Alex-D/@Alex-Dhttps://zestedesavoir.com/forums/sujet/831/certaines-pr-sont-a-valider-par-les-admin-sys/?page=1#p11701<blockquote>
<p>plutot qu'en évol</p>
</blockquote>
<p>Ce sont de simples pages HTML… c'est juste une config avec ngnix : au lieu d'aller sur une page 500 actuellement, il faut mettre celle-là. Et pour la maintenance je ne connais pas ngnix mais ça doit pas être bien sorcier non plus.</p>Certaines PR sont à valider par les admin sys., message #114352014-07-25T13:57:23+02:00firm1/@firm1https://zestedesavoir.com/forums/sujet/831/certaines-pr-sont-a-valider-par-les-admin-sys/?page=1#p11435<p>Pour la première PR, j'ai fait certaine remarques à prendre en compte.</p>
<p>Pour la deuxième, il faudrait que ce soit tester sur la preprod posément, et je n'ai pas ce temps (ou plutôt je priorise mon temps ailleurs pour les bug plutot qu'en évol)</p>Certaines PR sont à valider par les admin sys., message #113222014-07-25T03:25:41+02:00Alex-D/@Alex-Dhttps://zestedesavoir.com/forums/sujet/831/certaines-pr-sont-a-valider-par-les-admin-sys/?page=1#p11322<p>Salut !</p>
<p>Je poste un message ici pour attirer l'attention des admin sys sur les PR :</p>
<ul>
<li><a href="https://github.com/zestedesavoir/zds-site/pull/1155">#1155 - Script de base pour déployer en prod</a> </li>
<li><a href="https://github.com/zestedesavoir/zds-site/pull/1147">#1147 - Ajout des pages de maintenance et d'erreur 500 pour nginx</a>.</li>
</ul>
<p>Ces PR ne sont testables que sur un environnement iso-prod et à valider par les gens qui vont s'en servir.</p>
<p>Elles sont en attente de puis 2 semaines et sont liées aux milestones RC ou v1.0. Il serait donc judicieux de les valider et les merger relativement rapidement, car j'imagine que si c'était dans ces milestones c'est que c'était important pour la sortie.</p>
<p>On mergeant la #1155 on pourrait clore la milestone RC. Et en mergeant a 1147 on pourrait avoir une page de maintenance et un page d'erreur 500 pour ngnix qui soient beaucoup plus sexy.</p>
<p>C'est tout pour moi <img alt=":)" src="/static/smileys/smile.png"></p>