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] PixAdmin : pouvoir dupliquer les contenus formatifs (PIX-16670)(PIX-16075) #11504

Merged
merged 6 commits into from
Feb 28, 2025

Conversation

dianeCdrPix
Copy link
Contributor

@dianeCdrPix dianeCdrPix commented Feb 24, 2025

🥞 Problème

Aujourd'hui, on ne peut pas dupliquer un contenu formatif depuis PixAdmin. Il faut le recréer entièrement.

🥓 Proposition

Ajouter un bouton "Dupliquer ce contenu formatif" sur le détail d'un CF, à l'image du bouton "Dupliquer ce profil cible" sur la page détails d'un PC.
Une modale doit s'ouvrir pour confirmer la duplication.
On veut afficher ensuite la page d'édition du nouveau CF créé. Si pas possible, afficher la page de détails du nouveau CF.

🧃 Remarques

  • Je me demande si c'est judicieux que la route POST trainings/:id/duplicate renvoie un objet trainingId plutôt que directement la valeur de l'id du nouveau contenu formatif

😋 Pour tester

  • Se connecter dans PixAdmin
  • Aller sur la page de détails de n'importe quel contenu formatif
  • Cliquer sur "Dupliquer ce contenu formatif"
  • Vérifier que la modale s'ouvre. Si oui, cliquer sur "Valider"
  • La modale disparaît. Une pop-up en bas à droite indique si la duplication a réussi ou non
  • La page de détails du CF dupliqué s'affiche à l'écran.

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

@dianeCdrPix dianeCdrPix self-assigned this Feb 24, 2025
@dianeCdrPix dianeCdrPix force-pushed the pix-16670-duplicate-training-from-admin branch from 174dbc3 to f539ce6 Compare February 25, 2025 11:00
@dianeCdrPix dianeCdrPix requested a review from a team as a code owner February 25, 2025 11:00
@dianeCdrPix dianeCdrPix force-pushed the pix-16670-duplicate-training-from-admin branch 5 times, most recently from 9e8fbc5 to 09bd415 Compare February 26, 2025 15:23
@dianeCdrPix dianeCdrPix force-pushed the pix-16670-duplicate-training-from-admin branch from 09bd415 to 5627160 Compare February 26, 2025 15:42
@dianeCdrPix dianeCdrPix changed the title [FEATURE] PixAdmin : pouvoir dupliquer les contenus formatifs (PIX-16670) [FEATURE] PixAdmin : pouvoir dupliquer les contenus formatifs (PIX-16670)(PIX-16075) Feb 26, 2025
@dianeCdrPix dianeCdrPix force-pushed the pix-16670-duplicate-training-from-admin branch 5 times, most recently from 172f0eb to e3902d5 Compare February 27, 2025 15:31
@dianeCdrPix
Copy link
Contributor Author

Je ne parviens pas à avoir la notif de succès sur la RA, mais je l'ai en local #caMarcheEnLocal

@dianeCdrPix dianeCdrPix force-pushed the pix-16670-duplicate-training-from-admin branch from e3902d5 to a4de75f Compare February 27, 2025 16:10
Copy link

@jujumathieu jujumathieu left a comment

Choose a reason for hiding this comment

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

Looks good to me 🚀

@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-16670-duplicate-training-from-admin branch from a4de75f to 304099d Compare February 28, 2025 09:34
@pix-service-auto-merge pix-service-auto-merge merged commit c0003e6 into dev Feb 28, 2025
8 of 11 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-16670-duplicate-training-from-admin branch February 28, 2025 09:41
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.

5 participants