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

Réorganise le module tutorialv2 #6591

Merged
merged 6 commits into from
Mar 29, 2024
Merged

Conversation

Arnaud-D
Copy link
Contributor

@Arnaud-D Arnaud-D commented Mar 9, 2024

C'est de la refacto à fonctionnalités égales.

L'idée est de grouper les vues avec les formulaires afférants pour différentes fonctionnalités (modification de la licence, des auteurs, des contributeurs, des tags...)

Contrôle qualité

La CI suffit théoriquement, mais un petit test rapide des différentes fonctionnalités est le bienvenu en complément.

Note : pour tester les suggestions avec Python >= 3.10, il faut patcher Elastic Search pour remplacer collections.Mappings par collections.abc.Mappings. Il suffit de se laisser guider par les erreurs au fur et à mesure quand on indexe, quand on lance Elastic Search et quand on utilise le formulaire des suggestions.

@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 Mar 9, 2024
@coveralls
Copy link

coveralls commented Mar 9, 2024

Coverage Status

coverage: 88.699% (+0.03%) from 88.671%
when pulling 6c0d604 on Arnaud-D:rangement
into 2da324a on zestedesavoir:dev.

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 6e0d801 into zestedesavoir:dev Mar 29, 2024
12 checks passed
@Arnaud-D Arnaud-D deleted the rangement branch March 29, 2024 20:58
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