-
Notifications
You must be signed in to change notification settings - Fork 91
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
amelioration(tuile.chorus): ETQ admin, je peux saisir le cadre budgetaire d'une demarche de subvention pour faciliter le rapprochement d'un export DS a un export Chorus #9420
Conversation
PR Summary
|
c441d04
to
c478fee
Compare
bfd6961
to
c4f1c7e
Compare
689318b
to
faf086f
Compare
…`chorus` (jsonb)
…d' du ChorusConfiguration
…pi chorus fournies par la bretagne
…ctionnel, referentiel de programmation
…ette fonction est active sur la procedure
… ci n'est que partiellement rempli
…echerche ne fait pas plus de 2 caractères
a5fb5e1
to
ba6819c
Compare
…e quand il n'y a pas de resultat, change l'implem pour ce cas
…onfusion avec ActiveRecord.types.json
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
quelques retours nommages et cosmétiques, sinon dans le fond c'est bon pour moi ! et c'est trop cool pour la lisibilité d'avoir des concerns et petites responsabilités
app/components/procedure/chorus_form_component/chorus_form_component.html.haml
Outdated
Show resolved
Hide resolved
95acf75
to
13fe7f2
Compare
… soumettre à l'API, et quelques maj de style Co-authored-by: Colin Darie <colin@darie.eu>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀
ebea269
issue: #7395
pour rappel, la cible court terme est :
PS: en cible plus long terme, on souhaite pouvoir informer l'usager de l'etat d'avancement de sa subvention (afin d'eviter aux services d'instructeur d'etre appelé X fois pour savoir ou en est la demande de subvention)
le reste à faire (pour atteindre la cible) :
Cette PR intègre les premiers pas pour atteindre la cible :
ETQ SuperAdmin, je souhaite pouvoir activer la feature pour les beta-testeurs :
il y a un feature flag qui permet ou pas d'activer la tuile pour une procedure
feature_enabled?(:chorus)
ETQ admin, ayant la feature activé, je trouve la tuile de configuration du cadre budgétaire Chorus dans les options avancé de configuration d'une démarche :
ETQ admin, je peux renseigner le cadre budgetaire Chorus en utilisant les API de la bretagne
concernant les endpoints et la doc :
Suite au cadrage de sopra (Mr Philippon & je sais plus), le cadre budgetaire inclus ±7/8 champs. Cependant il suffit d'en demander 3 pour faciliter le rapprochement DS<=>Chorus.
Visuellement ça donne ça :
PS: un gros bravo a @tchak pour ce composant d'autocomplete qui a été facile d'usage.
PPS: @tchak , si t'as un instant, je suis pas certains du bon niveau d'encapsulation des
Success
/Failure
de mon implem. Si t'as un temps pour pairer et m'exposer ton avis, je suis preneur