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

WIP – ETQ administrateur je peux configurer un champ référentiel à configurer (avancé) #11202

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

mfo
Copy link
Contributor

@mfo mfo commented Jan 13, 2025

progress: #11161

tldr

Cette PR intègre une partie du wizard de configuration du type de champ référentiel à configuré avancé

  1. gère la logique d'"immutabilité" -> en gros on edit pas un referentiel qui tourne en prod
  2. gère le stockage / querying de l'API -> le plus simplement du monde (cible, RNB) : aka : pas d'auth, du get, rien de ouf
  3. permet de stocker sur le tdc le mapping api <-> json value

La suite :

  • implementer l'ui côté usager, parce que mine de rien on commence a stocker les infos de requete d'api
  • implementer la verbalisation des changement (les modification du formulaire usager) et la validation de ce type de champ (on ne publie pas une demarche avec un referentiel externe a configurer si celui ci n'est pas bien configuré)
  • revoir la logique de gestion des erreurs (ETQ usager) des champs cablés sur api, cf : WIP – correctif: Le champ rna peut rendre deux message de status #11028

ETQ admin, je suis invité a configurer le type de champ référentiel à configurer (avancé)

Capture d’écran 2025-01-15 à 3 19 11 PM quand le référentiel est bien configuré (on arrive a appeler l'API) Capture d’écran 2025-01-15 à 3 18 42 PM quand le référentiel n'est pas bien configuré (todo ou on a pas 200)

A noter :

  • pour une demarche en brouillon, je peux modifier un référentiel deja configuré
  • pour une demarche publiée, je suis amené sur un clone du référentiel déjà configuré
  • pour une demarche clonée, je suis amené sur un clone du référentiel déjà configuré sur l'autre démarche

ETQ admin, je peux configurer un référentiel de type API à configurer (avancé)

Screenshot 2025-01-15 at 15-22-42 demarches-simplifiees fr
écran par défault, les futures options sont pour le moment présentes, mais non implementées et disabled pour celles qui ne sont pas sur le chemin critique du RNB (donc par de configuration par csv, pas d'autocomplete)

Screenshot 2025-01-15 at 15-24-11 demarches-simplifiees fr
etq administrateur je peux saisir l'url de l'api, des données pour tester l'api, un hint pour guider l'utilisateur

ETQ administrateur, si je configure un référential de type API, si la config est mauvaise. Ay Caramba !

Screenshot 2025-01-15 at 15-27-49 demarches-simplifiees fr

ETQ administateur, si j'ai bien configuré mon champ, tada

Screenshot 2025-01-15 at 15-32-41 demarches-simplifiees fr
j'ai accès a la payload

Screenshot 2025-01-15 at 15-33-36 demarches-simplifiees fr

  1. on liste les clé du json au format jsonpath
  2. on affiche un exemple de valeur
  3. on propose un type par défaut se basant sur l'exemple
  4. on permet de selectionner ou pas que le champ sera utilisé pour le prefill (not yet implemented) et, si pas de prefill, on peut nommer le champ

@mfo mfo force-pushed the US/feat-champ-referentiel-mapping branch 4 times, most recently from c8575b1 to cf4b4c8 Compare January 13, 2025 15:54
Copy link

codecov bot commented Jan 13, 2025

Codecov Report

Attention: Patch coverage is 24.25743% with 153 lines in your changes missing coverage. Please review.

Project coverage is 60.24%. Comparing base (6ca8ddb) to head (418e1af).

Files with missing lines Patch % Lines
...rollers/administrateurs/referentiels_controller.rb 0.00% 65 Missing ⚠️
app/components/referentiels/new_form_component.rb 0.00% 35 Missing ⚠️
...ypes_de_champ_editor/info_referentiel_component.rb 0.00% 28 Missing ⚠️
app/services/referentiel_service.rb 0.00% 23 Missing ⚠️
.../components/referentiels/mapping_form_component.rb 95.55% 2 Missing ⚠️

❗ There is a different number of reports uploaded between BASE (6ca8ddb) and HEAD (418e1af). Click for more details.

HEAD has 11 uploads less than BASE
Flag BASE (6ca8ddb) HEAD (418e1af)
16 5
Additional details and impacted files
@@             Coverage Diff             @@
##             main   #11202       +/-   ##
===========================================
- Coverage   84.49%   60.24%   -24.25%     
===========================================
  Files        1200     1205        +5     
  Lines       26358    29372     +3014     
  Branches     4962     4534      -428     
===========================================
- Hits        22270    17694     -4576     
- Misses       4088    11678     +7590     

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

@mfo mfo force-pushed the US/feat-champ-referentiel-mapping branch 4 times, most recently from bde6377 to dedb941 Compare January 15, 2025 13:33
@mfo mfo changed the title wip – Us/feat champ referentiel mapping ETQ administrateur je peux configurer un champ référentiel à configurer (avancé) Jan 15, 2025
@mfo mfo changed the title ETQ administrateur je peux configurer un champ référentiel à configurer (avancé) WIP – ETQ administrateur je peux configurer un champ référentiel à configurer (avancé) Jan 15, 2025
@mfo
Copy link
Contributor Author

mfo commented Jan 15, 2025

@tchak – je me suis battu ac mes PR, mais pr répondre a ta question pourquoi stocker le mapping sur le TDC ->

  1. Referentiel.has_many :types_de_champ, de fait le mapping est plutot sur le tdc non ?
  2. aussi, pr les futurs columns, ça evitera une requete sur le Referentiel.

@mfo mfo force-pushed the US/feat-champ-referentiel-mapping branch from dedb941 to 4b52a59 Compare January 15, 2025 15:25
@mfo mfo force-pushed the US/feat-champ-referentiel-mapping branch from 4b52a59 to 418e1af Compare January 15, 2025 15:44
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