Allow others API's non FC to use modalitites block

[Isalafont]

Need to implement this method to allow others api who does not have France connect to use modalities block

[linear]

DAT-822 Make modalities work for non FC apis

[skelz0r]

https://github.com/etalab/data_pass/blob/develop/app/models/authorization_request/api_particulier.rb#L5 ..?

[skelz0r]

Pourquoi ?

[skelz0r]

Par ailleurs, même si on avait besoin de cette méthode ça n'a pas de sens, il aurait fallu fixer car c'est un bug à mon sens.

[Isalafont]

Pourquoi ?

Parcequ'en integrant api ficoba, with_france_connect? génère une erreur en arrivant sur le bloc modalities.
Dans l'idéal il faudrait faire de la refacto dans le bloc modalities qui comprends une condition d'affichage avec le unless with_france_connec? nécéssaire pour api impot particulier.
J'aimerais juste pouvoir avancer sur l'intégration de cette API

[skelz0r]

Ok donc c'est la vue le problème si je comprends bien.

[jbfeldis]

Ouaip, pareil pour R2P.

Valentin a ajouté cette méthode dans le concern spécifique à dgfip/fc sauf qu'elle est appelée dans un template utilisé potentiellement par d'autres types de forms.
A mon sens c'est bien d'avoir cette méthode (parce que FC c'est quand même lié à plus de choses que des modalités d'accès simples) mais du coup il y a un trou dans la raquette parce qu'elle n'est pas disponible sur toutes les AR qui ont des modalités.

[skelz0r]

Ok, donc le bug est que app/views/authorization_request_forms/blocks/default/_modalities.html.erb devrait être une vue spécifique à l'API Impot Particulier (ou toute API ayant une modalité FC, soit les API FC), et faut rendre la vue modalités plus générique.

[jbfeldis]

Exact, donc on pourrait utiliser le système de surcharge mais il y a beaucoup de forms donc ça ferait pas mal de dossiers/partials à rajouter (ou itérer pour surcharger "mieux")

[skelz0r]

#668

[Isalafont]

Du coup on fait quoi ?
on merge cette PR pour nous laisser avancer sur l'integration des dernières API DGFiP et en créant un autre ticket pour améliorer la vue des modalitées et supprimer l'erreur ultérieurement ?

[skelz0r]

J'ai à priori fini la PR.
Tu peux faire ton hack pour avancer sur ta branche, et rebase/fixer dès que c'est mergé sur la 668

[skelz0r]

(parce que le fix de cette PR ne fixera pas la vue dtf donc tu aurais eu le souci plus loin)

[Isalafont]

Je comptais faire la refacto une fois la PR de l'intégration de ficoba terminée 😅