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

Découple le module de forum du module de notification #5976

Merged

Conversation

Arnaud-D
Copy link
Contributor

@Arnaud-D Arnaud-D commented Oct 18, 2020

C'est une suite de #5971. Allez lire là-bas pour des éléments de contexte.

Cette PR est une deuxième étape, qui s'occupe du module de forum. Les autres viendront plus tard.

On a ici :

  • la création de signaux directement dans le module de forum pour remplacer ceux précédemment importés du module de notification ;
  • la mise à jour du module de notification pour écouter les signaux du module de MP au lieu de ses propres signaux comme auparavant ;
  • la mise à jour des tests existants pour vérifier le bon envoi des signaux.

Je ne touche dans le module de forum à ce qui a trait aux abonnements à des forums ou sujets, parce que c'est normal que la vue ait connaissance à la fois du module de forum et du module de notifications. C'est surtout côté modèles qu'une bonne séparation a plus de sens selon moi.

Il y a aussi des classes partagées que je garde pour plus tard (classe Comment en particulier), ce sera dans un des derniers morceaux de refactorisation.

Contrôle qualité

  • Vérifier que toutes les notifications marchent correctement pour les forum. Liste que j'espère exhaustive :
    • réponse à des sujets,
    • abonnements à des sujets et nouveaux message dans ceux-ci,
    • abonnement à des forums et nouveaux sujets dans ceux-ci,
    • bonne disparition des notifications en cas de mise en corbeille,
    • marquage non-lu de messages.
  • Évidemment les test unitaires.

@coveralls
Copy link

coveralls commented Oct 18, 2020

Coverage Status

Coverage increased (+0.005%) to 86.855% when pulling e135260 on Arnaud-D:rendre_à_César_ce_qui_est_à_César_2 into 8902b9f on zestedesavoir:dev.

@Arnaud-D Arnaud-D force-pushed the rendre_à_César_ce_qui_est_à_César_2 branch from c2d2a8e to 26342ab Compare October 21, 2020 20:56
@Arnaud-D Arnaud-D marked this pull request as ready for review October 21, 2020 21:30
@Arnaud-D Arnaud-D added the C-Back Concerne le back-end Django label Oct 21, 2020
@Arnaud-D Arnaud-D force-pushed the rendre_à_César_ce_qui_est_à_César_2 branch from dd77550 to 63295e8 Compare December 9, 2020 21:14
@Situphen
Copy link
Member

@Arnaud-D Est-ce que tu aurais le temps de mettre à jour ta branche par rapport à upstream/dev et résoudre les conflits pour que je puisse faire la QA ? :)

@Arnaud-D Arnaud-D force-pushed the rendre_à_César_ce_qui_est_à_César_2 branch from 63295e8 to 1634fe2 Compare December 17, 2020 21:45
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 !

J'ai testé les différentes notifications du forum et tout fonctionne comme sur des roulettes !

@Situphen Situphen merged commit 9324f6e into zestedesavoir:dev Dec 17, 2020
@Arnaud-D Arnaud-D deleted the rendre_à_César_ce_qui_est_à_César_2 branch December 17, 2020 23:04
viki53 pushed a commit to viki53/zds-site that referenced this pull request Jul 7, 2022
…5976)

* Topic moved

* Topic edited

* Topic read and post read

* Post unread

* Fix argument name

* Try again!

* Fix shadowed function

* Fix self signaling in notifications

* Et de un

* Et de deux

* Lint

Co-authored-by: Situphen <Situphen@users.noreply.github.com>
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
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants