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

Mise à jour de python-slugify et django-uuslug #6257

Merged
merged 2 commits into from
Mar 25, 2022

Conversation

Situphen
Copy link
Member

@Situphen Situphen commented Mar 6, 2022

Mise à jour de python-slugify et django-uuslug

J'ai regardé les changements pour chacune des dépendances et il y a peu de changements dans le code, c'est principalement un passage de Travis à Github Actions ainsi que un changement des versions de Python et Django supportées. Cela dit, étant donné la sensibilité des slugs pour nous, j'ai préféré créer une PR à part du reste pour que l'on puisse s'assurer que tout fonctionne correctement.

QA :

  • source zdsenv/bin/activate && make update && make zmd-start && make run-back
  • Jouer avec les titres de plusieurs contenus homonymes pour vérifier qu'il n'y a pas de soucis avec les slugs

@coveralls
Copy link

coveralls commented Mar 6, 2022

Coverage Status

Coverage remained the same at 87.32% when pulling 9944445 on Situphen:slug into 48ca52f on zestedesavoir:dev.

@Situphen Situphen added this to En développement in Suivi des PR via automation Mar 6, 2022
@Situphen Situphen marked this pull request as ready for review March 6, 2022 14:49
@Situphen Situphen moved this from En développement to En attente de QA in Suivi des PR Mar 6, 2022
@Situphen Situphen added the C-Back Concerne le back-end Django label Mar 6, 2022
Suivi des PR automation moved this from En attente de QA to Fusionnable après rebase Mar 25, 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 ✔️

Testé en créant un billet avec pour titre Billet A1èéà et un avec Billet A1eea : le slug est bien suffixé avec -1 pour le deuxième billet.

@philippemilink philippemilink merged commit 326a0dd into zestedesavoir:dev Mar 25, 2022
Suivi des PR automation moved this from Fusionnable après rebase to Fusionnée Mar 25, 2022
@Situphen Situphen deleted the slug branch March 25, 2022 17:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Back Concerne le back-end Django
Projects
Archived in project
Suivi des PR
  
Fusionnée
Development

Successfully merging this pull request may close these issues.

None yet

3 participants