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

Toilettage des vues des MP #6404

Merged
merged 2 commits into from
Oct 25, 2022
Merged

Toilettage des vues des MP #6404

merged 2 commits into from
Oct 25, 2022

Conversation

Arnaud-D
Copy link
Contributor

@Arnaud-D Arnaud-D commented Oct 19, 2022

C'est une série de refactorisation du code des MP. Il n'y a pas de gros changements, vraiment une somme de petits changements dans chacune des vues, d'où le terme de "toilettage".

Contrôle qualité

Les tests automatisés devraient faire une bonne partie du boulot.

Sinon, on peut tester rapidement les différentes fonctionnalités gérées par les vues modifiées :

  • la liste des MP,
  • la création d'un MP,
  • édition d'un MP (en tant qu'auteur, pas l'édition du message),
  • quitter un MP,
  • quitter plusieurs MP d'un coup depuis la liste,
  • l'affichage des messages d'un MP,
  • répondre à un MP,
  • éditer un message dans un MP,
  • marquer un MP comme non lu,
  • et c'est déjà pas mal !

@Arnaud-D Arnaud-D added the C-Back Concerne le back-end Django label Oct 19, 2022
@Arnaud-D Arnaud-D marked this pull request as draft October 19, 2022 20:20
@Arnaud-D Arnaud-D force-pushed the refacto-mp branch 3 times, most recently from 981e4e6 to b011cfb Compare October 19, 2022 22:14
@coveralls
Copy link

coveralls commented Oct 19, 2022

Coverage Status

Coverage decreased (-0.01%) to 88.306% when pulling fdf5561 on Arnaud-D:refacto-mp into d5dfbc6 on zestedesavoir:dev.

@Arnaud-D Arnaud-D marked this pull request as ready for review October 19, 2022 23:02
@Arnaud-D Arnaud-D added the hacktoberfest-accepted Pull request approuvée pour le Hacktoberfest label Oct 20, 2022
Copy link
Member

@Situphen Situphen 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 C'est en effet plus propre qu'avant !

@Situphen Situphen enabled auto-merge (rebase) October 25, 2022 17:26
@Situphen Situphen merged commit 491b135 into zestedesavoir:dev Oct 25, 2022
@Arnaud-D Arnaud-D deleted the refacto-mp branch October 26, 2022 06: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 hacktoberfest-accepted Pull request approuvée pour le Hacktoberfest
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants