Skip to content

[TECH] Déplacer la route des contenus formatifs vers le domaine Dev Comp (PIX-16249). #11285

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

Merged
merged 1 commit into from
Feb 18, 2025

Conversation

Libouk
Copy link
Member

@Libouk Libouk commented Jan 30, 2025

🥞 Problème

Dans le cadre de la migration DDD, on souhaite migrer les routes restantes vers src.

🥓 Proposition

Bien que consommés dans un contexte De Profil cible, les contenus formatifs accessibles depuis cette route sont liés aux "trainings" et c'est pourquoi j'ai migré la route vers ce domaine plutôt que de créer un nouveau domaine spécifique.

😋 Pour tester

  • Se connecter à Pix Admin (par exemple avec superadmin@example.net)
  • Aller sur la page Profils cibles
  • Ouvrir un Profil cible ayant des contenus formatifs (par exemple [Pix+Édu 1D FC] Prêt pour la certification du volet 1)
  • Aller sur l'onglet Contenus formatifs
  • Vérifier que ceux-ci s'affichent
  • Tldr: Cette page contient des résultats

@Libouk Libouk added 👀 Tech Review Needed 👀 Func Review Needed Need PO validation for this functionally team-evaluation PR relatives à l'expérience d'évaluation team-devcomp labels Jan 30, 2025
@Libouk Libouk self-assigned this Jan 30, 2025
@Libouk Libouk requested a review from a team as a code owner January 30, 2025 15:48
@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 :

@Libouk Libouk closed this Jan 31, 2025
@Libouk Libouk reopened this Jan 31, 2025
@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 :

@Libouk Libouk closed this Feb 3, 2025
@Libouk Libouk reopened this Feb 3, 2025
@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 :

Copy link
Contributor

@er-lim er-lim left a comment

Choose a reason for hiding this comment

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

Test en RA ✅

Capture d’écran 2025-02-04 à 09 10 11

@Libouk Libouk force-pushed the pix-16249-migrate-training-summaries branch from ca63f90 to c03dd61 Compare February 5, 2025 08:39
Copy link
Contributor

@er-lim er-lim left a comment

Choose a reason for hiding this comment

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

lu et testé en RA ✅

@yaf yaf added this to the Bounded context migration milestone Feb 6, 2025
@Libouk Libouk force-pushed the pix-16249-migrate-training-summaries branch from c03dd61 to 66a75ef Compare February 18, 2025 15:38
@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 :

@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-16249-migrate-training-summaries branch from 66a75ef to a919237 Compare February 18, 2025 15:41
@Libouk Libouk force-pushed the pix-16249-migrate-training-summaries branch from a919237 to 8526a0c Compare February 18, 2025 15:53
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-16249-migrate-training-summaries branch from 8526a0c to b40ede5 Compare February 18, 2025 15:53
@pix-service-auto-merge pix-service-auto-merge merged commit ddbcc6d into dev Feb 18, 2025
9 of 11 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-16249-migrate-training-summaries branch February 18, 2025 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants