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] 🚚 🧹 Déplacement et nettoyage du contenu du répertoire lib/domain/events #11167

Open
wants to merge 11 commits into
base: dev
Choose a base branch
from

Conversation

yaf
Copy link
Member

@yaf yaf commented Jan 21, 2025

🥞 Problème

Il y a plusieurs événements utilisé dans src qui sont encore dans lib

🥓 Proposition

Déplacer les événements restant vers src

🧃 Remarques

😋 Pour tester

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

@yaf yaf force-pushed the migrate-event-index-to-src branch 2 times, most recently from fb6bea7 to 1478daf Compare January 21, 2025 10:11
@xav-car
Copy link
Contributor

xav-car commented Jan 21, 2025

question
Il me semblait que le but des event ce n'était pas de les migrer tels quel mais de les supprimer au profit de job async ?

@yaf yaf force-pushed the migrate-event-index-to-src branch from 1478daf to 54a5b25 Compare January 21, 2025 11:31
@yaf
Copy link
Member Author

yaf commented Jan 22, 2025

Il me semblait que le but des event ce n'était pas de les migrer tels quel mais de les supprimer au profit de job async ?

Sans doute. Mais j'essaie d'avancer sur un point avant d'attaquer l'autre :)
La migration vers src remonte à 2 techdays, alors que les event n'ont qu'un techdays d'ancienneté il me semble :p

@yaf yaf force-pushed the migrate-event-index-to-src branch 2 times, most recently from 9e61f5e to 1e08989 Compare January 22, 2025 14:18
@yaf yaf force-pushed the migrate-event-index-to-src branch from 778644f to c0c559d Compare January 24, 2025 13:01
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.

3 participants