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

Enhance start habilitation #69

Merged
merged 24 commits into from
Jan 30, 2024
Merged

Conversation

skelz0r
Copy link
Member

@skelz0r skelz0r commented Jan 29, 2024

  • Enhance le authorization_requests#new design (changement orga design)
  • Gérer avec use_case -> filtrer directement
  • Gérer si éditeur -> rien => show disclaimer
  • Gérer avec un form uid direct (exemple de MGDIS)
  • Passer sur du full cucumber pour le decision tree (plus simple à lire)
  • Review

Closes https://linear.app/pole-api/issue/API-2117/page-de-demarrage-dhabilitation

It's a 2 step process now:

1. Organization verification
2. Choose the form

We can customize for each authorization the second page.
The second page is optional if there's only one form.
Still WIP, can be fixed later
@skelz0r skelz0r self-assigned this Jan 29, 2024
@skelz0r skelz0r force-pushed the features/enhance-start-habilitation branch 3 times, most recently from 905a034 to 433358a Compare January 29, 2024 10:52
Will be used in decision tree
Put default as last element for definition.
tags will be used within views for decision trees, it is not an actual model attribute
* Share complexe config between rspec and cucumber for docker/CI tests
* Harden web steps too, not so efficient with basic capybara features
* Introduce stimulus controller to handle the basic decision tree
  => Use tags to filter forms (thanks to previous commits)
* New card decision for start (use a data-turbo-method, which is not
  a so great solution, but hey it works with the DSFR)
* Activate js in cucumber to be sure that is works (without js it works
  too because it does not handle fr-hidden well with rack test)
@skelz0r skelz0r force-pushed the features/enhance-start-habilitation branch from 709a62b to 2c664be Compare January 29, 2024 16:55
@skelz0r skelz0r force-pushed the features/enhance-start-habilitation branch from 2c664be to 34636cf Compare January 29, 2024 18:00
@skelz0r skelz0r force-pushed the features/enhance-start-habilitation branch from f9ed1e5 to 4475123 Compare January 29, 2024 19:04
Copy link
Contributor

@Un3x Un3x left a comment

Choose a reason for hiding this comment

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

Pas grand chose à dire, des petites remarques mais en vrai c'est pas nécessaire.

@skelz0r skelz0r force-pushed the features/enhance-start-habilitation branch from 93f1b9a to ebe70a6 Compare January 30, 2024 14:02
@skelz0r skelz0r force-pushed the features/enhance-start-habilitation branch from ebe70a6 to 65a6bfc Compare January 30, 2024 14:07
@skelz0r skelz0r merged commit f530956 into develop Jan 30, 2024
7 checks passed
@skelz0r skelz0r deleted the features/enhance-start-habilitation branch January 30, 2024 14:12
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.

3 participants