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

Rassemble les intégrations Litteralis en une seule #1146

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

Conversation

florimondmanca
Copy link
Collaborator

@florimondmanca florimondmanca commented Jan 14, 2025

Cette PR regroupe les 3 intégrations Litteralis (MEL, Fougères, Lons-le-Saunier) en une seule intégration Litteralis, où les organisations et leurs identifiants d'API sont entièrement configurables par des variables d'environnement

  • => Plus besoin d'ajouter du code pour intégrer les données d'une collectivité (sauf si adaptation nécessaire de la logique d'intégration)
  • => Réduit le temps d'embarquement de la collectivité 🕐

Comment ça marche ? => Lire la nouvelle doc

TODO

  • Màj doc
  • Tests env var processor
  • Màj autres tests
  • Variable APP_LITTERALIS_ENABLED_ORGS dans GitHub Actions : ["mel", "fougeres", "lons_le_saunier"]
  • Secret APP_LITTERALIS_IMPORT_DATABASE_URL
  • Secrets *_ID et *_CREDENTIALS pour chaque org dans GitHub Actions

@florimondmanca florimondmanca force-pushed the refactor/litteralis-unified branch from b07ae1d to 343ad68 Compare January 14, 2025 15:20
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Je supprime cette commande car elle n'était pas utile / utilisée

@codecov-commenter
Copy link

codecov-commenter commented Jan 14, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.83%. Comparing base (e26ee17) to head (e55b28e).

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #1146      +/-   ##
============================================
- Coverage     98.86%   98.83%   -0.04%     
- Complexity     1848     1854       +6     
============================================
  Files           370      367       -3     
  Lines          7962     7978      +16     
============================================
+ Hits           7872     7885      +13     
- Misses           90       93       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@florimondmanca florimondmanca force-pushed the refactor/litteralis-unified branch from 343ad68 to 1d51eb7 Compare January 15, 2025 11:04
tools/ci_litteralis_init_org_env_vars.py Dismissed Show dismissed Hide dismissed
@florimondmanca florimondmanca force-pushed the refactor/litteralis-unified branch from 1d51eb7 to 0c52edc Compare January 15, 2025 11:10
tools/ci_litteralis_init_org_env_vars.py Dismissed Show dismissed Hide dismissed
@florimondmanca florimondmanca force-pushed the refactor/litteralis-unified branch 2 times, most recently from 345043d to 738fca4 Compare January 15, 2025 13:25
@florimondmanca florimondmanca marked this pull request as ready for review January 15, 2025 15:08
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.

Regrouper les intégrations Litteralis
2 participants