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

[FEATURE] Empêche un prescrit de partager une participation à une campagne supprimée (PIX-12691) #9280

Conversation

machestla
Copy link
Contributor

@machestla machestla commented Jun 14, 2024

🦄 Problème

Dans cette epix de suppression des campagnes, nous allons flag les campagnes en BDD mais les prescrit vont continuer d’avoir accès aux campagnes.

🤖 Proposition

Nous souhaitons le même comportement que lorsqu’une campagne est archivée : le prescrit ne peut pas partager sa participation à une campagne supprimée.

  • Vérifier si besoin de migrer le endpoint
  • Protéger le endpoint de partage de résultats
  • Ne pas afficher le bouton de partage de résultat sur la page de fin de parcours (réutiliser le mécanisme de l'archivage)
  • Il s'agit de modifier un if dans le front
  • Faire remonter les infos de deletedAt/deletedBy dans le front (ou vérifier si y a pas une propriété can share déjà envoyé par le back)

🌈 Remarques

💯 Pour tester

  • commencer une participation à une campagne
  • supprimer la campagne
  • essayer d'envoyer ses résultats
  • Observer l'absence de bouton
  • Être renvoyé sur la page"Oups"
  • 🎉 🐕

@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

@machestla machestla force-pushed the pix-12691-prevent-from-sharing-results-to-deleted-campaign branch from 81a5b13 to 3490a15 Compare June 19, 2024 15:57
@machestla machestla marked this pull request as ready for review June 19, 2024 15:57
@machestla machestla requested a review from a team as a code owner June 19, 2024 15:57
@@ -70,6 +75,13 @@ class Campaign {
isArchived() {
Copy link
Contributor

Choose a reason for hiding this comment

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

question à vérifier si c'est encore utilisé

Copy link
Contributor Author

@machestla machestla Jun 20, 2024

Choose a reason for hiding this comment

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

Il est encore utilisé pour afficher ou pas des boutons d'actions sur une campagne et afficher ou pas une bannière avec un bouton pour désarchiver

@machestla machestla force-pushed the pix-12691-prevent-from-sharing-results-to-deleted-campaign branch 8 times, most recently from 882e57a to 0456d54 Compare June 21, 2024 15:30
@lionelB lionelB added Func Review OK PO validated functionally the PR and removed 👀 Func Review Needed labels Jun 24, 2024
Copy link
Contributor

@lionelB lionelB left a comment

Choose a reason for hiding this comment

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

Tech ok !

@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-12691-prevent-from-sharing-results-to-deleted-campaign branch 2 times, most recently from 9c7dd14 to 0112c27 Compare June 24, 2024 09:17
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-12691-prevent-from-sharing-results-to-deleted-campaign branch from 0112c27 to dc05796 Compare June 24, 2024 12:42
@pix-service-auto-merge pix-service-auto-merge merged commit cf05666 into dev Jun 24, 2024
6 of 7 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-12691-prevent-from-sharing-results-to-deleted-campaign branch June 24, 2024 12:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants