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

Corrige un bug de notification persistante de nouveau contenu d'un membre suivi #6491

Merged
merged 2 commits into from
Aug 27, 2023

Conversation

philippemilink
Copy link
Member

@philippemilink philippemilink commented Apr 15, 2023

Fix #5544

Contrôle qualité

Scénario 1

  1. L'utilisateur A suit l'utilisateur B
  2. B écrit avec l'utilisateur C un contenu qu'il publie
  3. A reçoit une notification d'un nouveau contenu de B, mais il ne clique pas dessus
  4. B se retire de la liste des auteurs du contenu
  5. A clique sur la notification du nouveau contenu. Résultat : la notification est marquée comme lue.

Scénario 2

  1. L'utilisateur A suit l'utilisateur B
  2. B écrit avec l'utilisateur C un contenu qu'il publie
  3. A reçoit une notification d'un nouveau contenu de B, mais il ne clique pas dessus
  4. B se retire de la liste des auteurs du contenu
  5. C modifie le contenu et publie la nouvelle version (sans B comme co-auteur)
  6. A clique sur la notification du nouveau contenu. Résultat : la notification est marquée comme lue.

Pour les deux scénarios : vérifier que B et C ont une notification pour signaler la publication de leur contenu (elle est automatiquement lue pour celui qui publie le contenu et non-lue pour l'autre co-auteur). Dans le deuxième scénario, B n'a pas de notification pour la deuxième publication du contenu par C.

@coveralls
Copy link

coveralls commented Apr 15, 2023

Coverage Status

coverage: 88.398% (+0.005%) from 88.393% when pulling a374dbb on philippemilink:mark-read-quit-authorship into b6fd744 on zestedesavoir:dev.

@Migwel
Copy link
Contributor

Migwel commented May 14, 2023

Le cas de base est effectivement résolu mais le cas suivant ne fonctionne toujours pas:

  1. On suit une personne.
  2. Cette personne publie un contenu avec un autre auteur
  3. On reçoit une notification pour ça, vu qu'on est abonné⋅e. Mais on ne clique pas dessus.
  4. Cette personne se retire des auteurs dans la version brouillon.
  5. L'autre auteur met à jour le contenu et le publie
  6. On clique sur la notification.

Comportement attendu:
La notification ne persiste pas

Comportement actuel:
La notification persiste

@philippemilink philippemilink force-pushed the mark-read-quit-authorship branch 2 times, most recently from de703bb to 55afb5f Compare June 3, 2023 19:42
@philippemilink
Copy link
Member Author

J'ai corrigé et mis à jour les instructions de QA.

Copy link
Contributor

@Arnaud-D Arnaud-D left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Scénario 1 OK
Scénario 2 OK

QA OK ✔️

@Arnaud-D Arnaud-D enabled auto-merge (squash) August 27, 2023 08:41
@Arnaud-D Arnaud-D merged commit b5c6af7 into zestedesavoir:dev Aug 27, 2023
8 checks passed
@philippemilink philippemilink deleted the mark-read-quit-authorship branch August 27, 2023 18:08
Migwel pushed a commit to Migwel/zds-site that referenced this pull request Sep 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Notification persistante liée à la publication d'un billet
4 participants