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

ETQ administrateur, je peux ajouter des conditions d'eligibilité auxquelles les dossiers doivent correspondre sans quoi l'usager ne peut déposer son dossier #10292

Merged
merged 12 commits into from
Jun 11, 2024

Conversation

mfo
Copy link
Contributor

@mfo mfo commented Apr 9, 2024

see: #10229

si ca peut aider pr la relecture, c'est pas si gros enfait : (± ajout 1000 lignes de test et 300 suppression)

TODO :

  • gerer les erreurs sur les conditions
  • gerer les conditions sur les repetitions
  • inverser la logique – on declare des règles d'ineligibilite
  • footer a la lisa
  • des couleurs plus douce sur les tuiles
  • tester en prod

une spec un peu textuel et visuel

ETQ admin, sur la page de gestion de ma démarche

  • j'ai accès a une tuile pour configurer l'inéligibilité des dossiers depuis la page de gestion de la démarche
Capture d’écran 2024-06-06 à 12 06 21 PM

ce n'est pas obligatoire, mais l'UX propose de co-localisé la tuile a côté de sa dépendance (et ça me logique)

ETQ admin, sur la page de gestion des critères d'inéligibilité

  • si ma démarche n'a pas de champs supportant les conditions, je suis invité à en ajouter un
Capture d’écran 2024-06-06 à 9 48 16 AM
  • je peux activer/desactiver/configurer les critères l'ineligibilité des dossiers sur ma démarche une fois un champs de conditionnel présent
Capture d’écran 2024-06-06 à 9 49 20 AM

ETQ admin, sur la page de gestion de ma démarche, après avoir configuré mes règles

  • je suis averti que la configuration des conditions l'inéligibilité des dossiers requiert de republier ma démarche
Capture d’écran 2024-06-06 à 4 56 20 AM-2
  • j'ai un resumé des changements concernant la configuration de l'inéligibilité des dossiers
Capture d’écran 2024-06-06 à 4 58 12 AM

ETQ admin, sur l'editeur de champs

  • je ne peux supprimer un champ utilisé pour l'éligibilité des dossiers
Capture d’écran 2024-05-15 à 3 44 56 PM

ETQ usager

  • tant qu'on ne peut pas déterminer de manière certaines que le dossier est inéligible (1 à n conditions, donc des règles avec des ET), je peux remplir mon forumlaire
Capture d’écran 2024-05-15 à 3 54 08 PM

(donc le formulaire ne bouge pas)

  • une fois l'inélégibilité garantie, une modale apparait avec le message d'inégibilité défini par l'admin
Capture d’écran 2024-05-17 à 4 00 48 PM
  • le bouton de dépot de dossier est désactivé et un lien pour re-afficher la modal est ajouté
Capture d’écran 2024-05-17 à 4 00 55 PM

(a noter qu'au rechargement de la page, le bouton d'inégibilité est présent et permet de ré-afficher la modal)

  • si je change mes réponses, je peux a nouveaux deposer mon dossier si celui ci répond aux critère d'éligibilité
Capture d’écran 2024-05-15 à 4 40 56 PM

@mfo mfo force-pushed the US/elligible-on-submit branch from a4e6423 to f3b6770 Compare April 25, 2024 04:44
@mfo mfo force-pushed the US/elligible-on-submit branch 2 times, most recently from 4ccab42 to 8988d6b Compare May 7, 2024 12:23
Copy link

codecov bot commented May 7, 2024

Codecov Report

Attention: Patch coverage is 96.19048% with 8 lines in your changes missing coverage. Please review.

Project coverage is 80.98%. Comparing base (080a818) to head (f819da8).
Report is 6 commits behind head on main.

Files Patch % Lines
.../procedure/card/ineligibilite_dossier_component.rb 77.77% 2 Missing ⚠️
.../administrateurs/ineligibilite_rules_controller.rb 94.73% 2 Missing ⚠️
app/models/procedure_revision_change.rb 94.73% 2 Missing ⚠️
app/models/concerns/champ_conditional_concern.rb 50.00% 1 Missing ⚠️
app/models/procedure_revision_type_de_champ.rb 50.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #10292      +/-   ##
==========================================
+ Coverage   80.88%   80.98%   +0.09%     
==========================================
  Files        1211     1217       +6     
  Lines       25639    25802     +163     
  Branches     4634     4659      +25     
==========================================
+ Hits        20739    20896     +157     
- Misses       4900     4906       +6     

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

@mfo mfo force-pushed the US/elligible-on-submit branch 8 times, most recently from e407f47 to 3f7dc49 Compare May 13, 2024 15:29
@mfo mfo force-pushed the US/elligible-on-submit branch 4 times, most recently from ec5b043 to 3334499 Compare May 15, 2024 13:33
@mfo mfo changed the title WIP – Us/elligible on submit ETQ administrateur, je peux ajouter des conditions d'eligibilité auxquelles les dossiers doivent correspondre sans quoi l'usager ne peut déposer son dossier May 15, 2024
@mfo mfo force-pushed the US/elligible-on-submit branch 9 times, most recently from 0e7c390 to b66676a Compare May 21, 2024 12:56
@mfo mfo force-pushed the US/elligible-on-submit branch 4 times, most recently from 800f89a to 37c6798 Compare May 23, 2024 12:45
@mfo mfo force-pushed the US/elligible-on-submit branch 7 times, most recently from 7e469be to d7c8b47 Compare June 7, 2024 14:26
Copy link
Member

@LeSim LeSim left a comment

Choose a reason for hiding this comment

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

yes, trop bien.

Pour moi c'est bon si tu fais sauter la partie la (grosse) partie computable? comme vu ensemble

.fr-toggle
= f.check_box :ineligibilite_enabled, class: 'fr-toggle__input', data: @opt
= f.label :ineligibilite_enabled, "Inéligibilité des dossiers", data: { 'fr-checked-label': "Actif", 'fr-unchecked-label': "Inactif" }, class: 'fr-toggle__label'
%p.fr-hint-text Passer l’intérrupteur sur activé pour que les critères d’inéligibilité configurés s'appliquent
Copy link
Member

Choose a reason for hiding this comment

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

"Activez l'interrupteur pour que les critères d’inéligibilité configurés s'appliquent" ?

@mfo mfo force-pushed the US/elligible-on-submit branch from d7c8b47 to de96670 Compare June 11, 2024 07:29
@mfo mfo force-pushed the US/elligible-on-submit branch from de96670 to e34db67 Compare June 11, 2024 07:40
@mfo mfo force-pushed the US/elligible-on-submit branch from e34db67 to f819da8 Compare June 11, 2024 07:57
@mfo mfo enabled auto-merge June 11, 2024 08:20
@mfo mfo added this pull request to the merge queue Jun 11, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jun 11, 2024
@mfo mfo added this pull request to the merge queue Jun 11, 2024
Merged via the queue into demarches-simplifiees:main with commit f6a5e93 Jun 11, 2024
19 checks passed
@mfo mfo deleted the US/elligible-on-submit branch June 11, 2024 09:53
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.

2 participants