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

[TECH] 🚚 deplace le learning-content-repository de lib ➡️ src (Pix-17008) #11661

Closed
wants to merge 2 commits into from

Conversation

lionelB
Copy link
Member

@lionelB lionelB commented Mar 12, 2025

🥞 Problème

on veut plus rien dans lib

🥓 Proposition

on déplace le repository

🧃 Remarques

c'etait un peu compliqué ar il y avait déja un fichier avec le meme nom mais on a décidé de le renommer en datasource car c'etait une classe absctraite utilisée pour intérroger les objet du learning content

😋 Pour tester

ras

@lionelB lionelB added 👀 Tech Review Needed 👀 Func Review Needed Need PO validation for this functionally cross-team Toutes les équipes de dev Bounded-context-migration labels Mar 12, 2025
@lionelB lionelB requested review from a team as code owners March 12, 2025 15:34
@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 :

@laura-bergoens
Copy link
Member

laura-bergoens commented Mar 12, 2025

Le sujet est très "contenus" en réalité. Il est prévu dans notre dette technique à dépiler au fur et à mesure de remettre dans le bon bounded context les répos et autres qui sont de notre scope.
Je voudrais d'abord en parler à l'équipe avant, mais comme ça je suis pas fan de ta PR juste parce que c'est à nous de le faire et qu'on a d'autres refactos aussi au passage
EDIT: j'ajouterai que pour déplacer de /lib vers /src/shared je vois pas trop l'intérêt du point de vue bounded context migration 🤷🏿

@lionelB
Copy link
Member Author

lionelB commented Mar 12, 2025

D'acc, du coup je ferme

@lionelB lionelB closed this Mar 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bounded-context-migration cross-team Toutes les équipes de dev 👀 Func Review Needed Need PO validation for this functionally 👀 Tech Review Needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants