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

Rend générique en fonction du type la vue "RemoveAuthorFromContent" #6668

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

Arnaud-D
Copy link
Contributor

Dans le cadre de la nouvelle organisation des contenus

  • supprime la distinction de type de publication
  • préfère le terme "publication" à "contenu"
  • renomme les vues pour adopter la convention de Django
  • refactorise pour améliorer la lisibilité
  • interdit les méthodes HTTP autres que POST

Contrôle qualité

Ajouter et supprimer des auteurs. Regarder les MP correspondants.

Supprimer aussi des contenus, vu que j'ai touché une bricole à ce niveau-là.

@Arnaud-D Arnaud-D added the C-Back Concerne le back-end Django label Oct 19, 2024
@coveralls
Copy link

coveralls commented Oct 19, 2024

Coverage Status

coverage: 89.217% (+0.01%) from 89.206%
when pulling 9f18af1 on Arnaud-D:retire-des-distinctions-entre-types-2
into b089a37 on zestedesavoir:dev.

@Arnaud-D Arnaud-D marked this pull request as ready for review October 19, 2024 07:09
* supprime la distinction de type de publication
* préfère le terme "publication" à "contenu"
* renomme les vues pour adopter la convention de Django
* refactorise pour améliorer la lisibilité
* interdit les méthodes HTTP autres que POST
@Arnaud-D Arnaud-D force-pushed the retire-des-distinctions-entre-types-2 branch from 30c2601 to 9f18af1 Compare November 11, 2024 10:51
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
Status: En attente de QA
Development

Successfully merging this pull request may close these issues.

2 participants