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

Feat: posthog informations transmition + data package refactos #1546

Merged
merged 18 commits into from
Jan 26, 2025

Conversation

ttdm
Copy link
Collaborator

@ttdm ttdm commented Jan 16, 2025

/!\ A ne pas merge, à review sans merge ! (350lignes à review,le reste c'est de la data)

Partie 1 de la PR : posthog

Je pense que c'est une PR intéressante à discuter avant review ou à review en pair.

L'idée est de créer des events posthogs liés aux envois de formulaires.
Les events posthogs étant non supprimable, la PR renvoie pour l'instant des console.log().
L'idée est de valider ces consoles.log() ensemble puis d'éxcuter une fois la PR "pour de vrai".
Cela permettra de créer une première série d'events.
Il faudra ensuite repasser en logs, pour vérifier qu'il n'y a pas de création d'events en doublon.
Une fois cela au point, on pourra valider la PR pour un passage en prod.

Remarques :

  • Sur le fichier data/src/common/posthog/posthogManager, la ligne, posthog.init ne fonctionne pas si l'import posthog est le suivant : import posthog, { PostHog } from 'posthog-js' ; il a fallu que je modifie cet import en import { posthog, PostHog } from 'posthog-js'. Le premier code, non fonctionnel ici, est pourtant présent dans le front. étonnant ?

Partie 2 de la PR : le refacto du package data :

  • suppression des dossiers generated au profit de static.

close #1244
close #901

@ttdm ttdm self-assigned this Jan 16, 2025
@dolemoine
Copy link
Collaborator

Première passe après échange avec @ttdm en début de journée, review de la refacto archi data et passe fonctionnelle sur l'app (server dev et build OK).
RAS de mon côté.
Non review : Manager & Updater

@dolemoine dolemoine self-requested a review January 22, 2025 11:13
@dolemoine dolemoine requested a review from yvalentin January 22, 2025 11:14
@ttdm ttdm linked an issue Jan 22, 2025 that may be closed by this pull request
@ttdm ttdm merged commit 14527a6 into main Jan 26, 2025
10 checks passed
@ttdm ttdm deleted the feat/posthog_transmit_brevo_status branch January 26, 2025 18:48
@yvalentin yvalentin removed their request for review January 27, 2025 14:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants