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

Refactorise les URL de la galerie #6293

Conversation

Arnaud-D
Copy link
Contributor

@Arnaud-D Arnaud-D commented May 6, 2022

Lié à #6246.

Au menu de cette refacto des URL de la galerie :

  • utilise un namespace,
  • utilise path au lieu de re_path,
  • renomme "new" en "create",
  • retire le slug pour les URLs de formulaire, il est conservé pour la vue "details".

Contrôle qualité

Vérifier que la galerie continue de fonctionner nominalement (juste exercer toutes les routes).

Vérifier que les liens ailleurs sont toujours bons (notamment côté tuto).

@Arnaud-D Arnaud-D added C-Back Concerne le back-end Django S-Refactorisation Améliore le code existant sans forcément ajouter de nouvelle fonctionnalité labels May 6, 2022
@coveralls
Copy link

coveralls commented May 6, 2022

Coverage Status

Coverage increased (+0.007%) to 88.001% when pulling d8adc10 on Arnaud-D:passez_votre_chemin_refacto_urls_en_cours_3 into 0b4c280 on zestedesavoir:dev.

@Arnaud-D Arnaud-D force-pushed the passez_votre_chemin_refacto_urls_en_cours_3 branch from 97907ca to e5c2f2b Compare June 2, 2022 21:48
* utilise un namespace
* utilise path au lieu de re_path
* renomme "new" en "create"
* retire le slug pour les URLs de formulaire, il est conservé pour la vue "details"
@Arnaud-D Arnaud-D force-pushed the passez_votre_chemin_refacto_urls_en_cours_3 branch from e5c2f2b to d959e85 Compare June 26, 2022 16:54
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.

QA OK ✔️

@philippemilink philippemilink merged commit dbf1e27 into zestedesavoir:dev Jul 3, 2022
@Arnaud-D Arnaud-D deleted the passez_votre_chemin_refacto_urls_en_cours_3 branch July 3, 2022 17:42
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 S-Refactorisation Améliore le code existant sans forcément ajouter de nouvelle fonctionnalité
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants