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

Retire la gestion des anciens smileys #6268

Merged
merged 2 commits into from
Mar 26, 2022

Conversation

Arnaud-D
Copy link
Contributor

@Arnaud-D Arnaud-D commented Mar 24, 2022

Fix #6263.

Contrôle qualité

  • Appliquer les migrations python3 manage.py migrate.
  • Vérifier que rien ne change et que tout fonctionne bien (les smileys et l'édition du profil notamment).

Mise en prod

@Arnaud-D Arnaud-D added this to En développement in Suivi des PR via automation Mar 24, 2022
@Arnaud-D Arnaud-D moved this from En développement to En attente de QA in Suivi des PR Mar 24, 2022
@Arnaud-D Arnaud-D moved this from En attente de QA to En développement in Suivi des PR Mar 24, 2022
@coveralls
Copy link

coveralls commented Mar 24, 2022

Coverage Status

Coverage decreased (-0.09%) to 87.226% when pulling 8895869 on Arnaud-D:no_country_for_old_smileys into 1a3bc43 on zestedesavoir:dev.

@Arnaud-D
Copy link
Contributor Author

Note : on perd en couverture de tests, mais c'est parce que ce qui teste le cookie pour les anciens smileys est en fait le seul test qui exécute la mise à jour de la page de profil. Le reste n'est pas réellement testé.

@Arnaud-D Arnaud-D moved this from En développement to En attente de QA in Suivi des PR Mar 25, 2022
@philippemilink
Copy link
Member

On peut donc aussi supprimer le dossier assets/smileys/old ?

@philippemilink philippemilink moved this from En attente de QA to En attente de retours in Suivi des PR Mar 25, 2022
@Arnaud-D
Copy link
Contributor Author

Je pense en effet qu'on peut le supprimer.

J'ai aussi ajouté une instruction pour la mise en prod, il y a des bricoles manuelles à faire si on veut nettoyer la vieille conf.

@Arnaud-D Arnaud-D moved this from En attente de retours to En attente de QA in Suivi des PR Mar 26, 2022
Suivi des PR automation moved this from En attente de QA to Fusionnable après rebase Mar 26, 2022
Copy link
Member

@philippemilink philippemilink left a comment

Choose a reason for hiding this comment

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

Rapport de QA

OK ✔️

Concernant la mise en production, on pourra supprimer ce fichier et cette ligne.

@philippemilink philippemilink merged commit b879b69 into zestedesavoir:dev Mar 26, 2022
Suivi des PR automation moved this from Fusionnable après rebase to Fusionnée Mar 26, 2022
@philippemilink
Copy link
Member

Concernant la mise en production, on pourra supprimer ce fichier et cette ligne.

J'ai parlé trop vite. La version HTML des commentaires et posts sur les forums étant stockée en base de données et le smiley ninja ayant changé d'extension, il faut conserver ce fichier, et juste l'adapter pour ne conserver que la gestion du smiley ninja. On pourra vérifier que les smileys ninja dans les vieux messages fonctionnent toujours avec ce message-ci, par exemple.

philippemilink added a commit to zestedesavoir/ansible-zestedesavoir that referenced this pull request Mar 26, 2022
Ce cookie n'est plus utilisé, voir
zestedesavoir/zds-site#6268 ou le commit
b879b69714c1268b19118383d3005509dcd9d62e de zds-site.
@Arnaud-D Arnaud-D deleted the no_country_for_old_smileys branch March 26, 2022 18:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Suivi des PR
  
Fusionnée
Development

Successfully merging this pull request may close these issues.

Supprimer la gestion des anciens smileys
3 participants