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 urls des Mise en une #6284

Conversation

Arnaud-D
Copy link
Contributor

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

Un des points de #6246 (mais qui mériterait probablement son propre ticket).

C'est une petite refactor de URLs. J'ai commencé par ce module, parce qu'il est simple. C'est le genre de choses où on aimerait bien avoir des tests à la Selenium pour éviter de casser des choses par accidents, d'ailleurs. ^^

Au menu :

  • utilisation de path (plus lisible et parfait niveau flexibilité pour notre usage) au lieu de re_path;
  • ça a pour conséquence d'utiliser les nouveaux motifs systématiquement au lieu de regex ;
  • ajout d'un namespace pour hiérarchiser les noms d'urls comme il faut ;
  • un petit changement de vocabulaire marginal éditer -> modifier.

Contrôle qualité

Jouer un peu avec le module de mise en Une / messages de page d'accueil. Le but est vraiment d'exercer les différentes routes, pas de tester le module en lui-même. Par exemple, on peut :

  • créer, supprimer des unes ou messages d'accueil ;
  • faire des requêtes de une, les ignorer, supprimer, etc.

@Arnaud-D Arnaud-D added the C-Back Concerne le back-end Django label Apr 6, 2022
@coveralls
Copy link

coveralls commented Apr 6, 2022

Coverage Status

Coverage increased (+0.0007%) to 87.994% when pulling fe7dcb4 on Arnaud-D:passez_votre_chemin_refacto_url_en_cours into 36550f7 on zestedesavoir:dev.

@Arnaud-D Arnaud-D added the S-Refactorisation Améliore le code existant sans forcément ajouter de nouvelle fonctionnalité label Apr 8, 2022
@Arnaud-D Arnaud-D force-pushed the passez_votre_chemin_refacto_url_en_cours branch from af7de1b to 4f9e763 Compare June 2, 2022 21:49
@Arnaud-D Arnaud-D force-pushed the passez_votre_chemin_refacto_url_en_cours branch from 4f9e763 to 3274d04 Compare June 26, 2022 16:50
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 enabled auto-merge (squash) July 3, 2022 15:31
@philippemilink philippemilink merged commit 0b4c280 into zestedesavoir:dev Jul 3, 2022
@Arnaud-D Arnaud-D deleted the passez_votre_chemin_refacto_url_en_cours branch July 3, 2022 15:38
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