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] Affiche la date de suppression d'une campagne sur PixAdmin (13054). #9318

Conversation

frinyvonnick
Copy link
Member

@frinyvonnick frinyvonnick commented Jun 19, 2024

🦄 Problème

Dans le cadre de l'Epix "Suppression de campagnes" les campagnes vont pouvoir être supprimées. On soupçonne que cela va engendré des questions au support. On a donc besoin de rendre accessible cette information facilement via PixAdmin.

🤖 Proposition

Ajouter dans la section Campagnes d'une organisation une colonne dans le tableau qui stipule si la campagne est supprimée et à quelle date.

🌈 Remarques

💯 Pour tester

  • En DB, mettre une date dans la colonne deletedAt d'une campagne
  • Se connecter à PixAdmin et aller sur la page de l'organisation dans la section campagnes
  • Constater la présence de la colonne "Supprimée le" et voir que la date est la bonne

@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 :

@lionelB
Copy link
Contributor

lionelB commented Jun 19, 2024

✔️ func ok

@@ -30,7 +30,7 @@ describe('Integration | Repository | Campaign-Management', function () {
const result = await campaignManagementRepository.get(campaign.id);

// then
expect(result).to.deep.equal({
Copy link
Contributor

Choose a reason for hiding this comment

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

question : quelle est la différence avec include ?

Copy link
Member Author

Choose a reason for hiding this comment

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

equal va casser si il y a une propriété en plus alors que include non :) Dans ce test, je ne voulais pas spécifier le deletedAt donc j'ai changé l'assertion pour un include.

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-13054-display-deleted-indicator-on-campaigns-in-admin branch from 39f670f to 492546d Compare June 20, 2024 13:04
@pix-service-auto-merge pix-service-auto-merge merged commit f1496c6 into dev Jun 20, 2024
6 of 7 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-13054-display-deleted-indicator-on-campaigns-in-admin branch June 20, 2024 13:12
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

4 participants