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(api): add new env sandbox #396

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Conversation

hlecuyer
Copy link
Contributor

@hlecuyer hlecuyer commented Mar 3, 2025

Lien vers le ticket notion associé

Check-list

  • Mes commits et ma PR suivent le conventional commit
    • <type>(<optional scope>): <description>
    • types : feat, chore, fix, docs, etc.
    • scopes : api, pipeline, deployment, deduplication, datawarehouse
  • Mes messages de commit sont en anglais
  • J'ai exécuté les pre-commits
  • J'ai indiqué le ticket notion associé
  • J'ai passé le ticket notion en review

si ma PR concerne la pipeline, le datawarehouse ou la deduplication

* [ ] J'ai actualisé les données liées à ma PR (avec une extraction récente)
* [ ] J'ai utilisé un dump récent de la prod
* [ ] J'ai exécuté en local les modèles (via dbt build -s models/... ou airflow en local)
* [ ] J'ai déployé en staging et exécuté les dags
* [ ] J'ai indiqué quelques questions metabase pour mettre en évidence l'impact de ma PR

si ma PR concerne l'api

  • J'ai réimporté les données marts depuis le datalake
  • J'emploie le français dans l'interface de l'api (query params, description, etc.)
  • Si ma PR inclut des modifs de l'orm, j'ai inclus les migrations alembic
  • J'ai ajouté des tests
  • J'ai fait une analyse perfs avant/après via locust

@hlecuyer hlecuyer force-pushed the feat/api/add-new-env-api branch from 8b9a3aa to 662dca7 Compare March 3, 2025 10:06
@hlecuyer hlecuyer requested a review from vperron March 3, 2025 10:14
@hlecuyer hlecuyer force-pushed the feat/api/add-new-env-api branch from 662dca7 to de29694 Compare March 3, 2025 10:32
@hlecuyer hlecuyer force-pushed the feat/api/add-new-env-api branch from de29694 to 5a75777 Compare March 3, 2025 10:33
@hlecuyer
Copy link
Contributor Author

hlecuyer commented Mar 3, 2025

L'idee est de rajouter une var d'env ANONYMIZE = true dans l'env de l'app scalingo.

Question: est ce que le trigger du deploiement de l'app sandbox doit ce faire depuis la branch release ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant