Draft: Hubee/dat 574/correctif wording #433

Isalafont · 2024-09-19T15:13:31Z

WIP : Reste à faire pour lundi :

Retirer le provider_name et appeler le provider.name dans la page d'unauthenticated_start pour spécifier le nom du fournisseur
Changer les nom des fournissuer pour hubee (DILA et DGS) + ajout des logos et urls correspondants
Modifier les logos s'affichant pour hubee
Ajouter un entête "Via HubEE" dans les formulaires adéquats.

linear · 2024-09-19T15:13:33Z

DAT-574 Changement des wordings sur les formulaires HubEE (CertDC et DILA)

skelz0r · 2024-09-19T15:57:18Z

Je suis désolé @Isalafont mais je me permets de faire un commentaire dès maintenant : il faut juste faire en sorte de modifier les wordings et non les vues, sinon on ne va jamais s'en sortir si on doit customiser pour tout le monde chaque vue avec des conditions partout, ça deviendra inmaintenable.

En regardant le figma, mise à part changer les "demandes d'habilitations" en "demandes d'abonnements" (qui pour moi est plus que douteux au passage, DataPass délivre des habilitations pas des "abonnements"), quasiment tout peut se faire en modifiant les wordings dans les fichiers YAML.

Ce que je te propose: tu modifies le maximum que tu peux dans les YAML, et ce que tu ne peux pas modifier sans avoir à faire des disjonctions on en discute de vive voix.

skelz0r · 2024-09-19T15:58:02Z

Ce que je te propose: tu modifies le maximum que tu peux dans les YAML, et ce que tu ne peux pas modifier sans avoir à faire des disjonctions on en discute de vive voix.

En ayant listé au préalable quelque part ofc.

skelz0r · 2024-09-20T10:58:45Z

Je me permets de mettre cette PR #437 qui va te permettre aussi de mieux customiser les vues en te basant sur les attributs des définitions et non sur le type spécifique de la définition.

Isalafont · 2024-09-23T14:08:11Z

Merci à toi @Samuelfaure pour le coup de main !! 🙏

app/models/authorization_request_form.rb

app/views/authorization_request_forms/new.html.erb

config/authorization_definitions.yml

app/models/authorization_definition.rb

evaspae · 2024-09-24T08:27:24Z

Pour les formulaires en single page, je propose de masquer la partie "les étapes de votre formulaire" car elle ne sert à rien.

On peut conserver la partie "les objectifs de ce formulaire" et y intégrer les wording demandés par HubEE.
Ça nous permet de régler le problème global d'info pas utile sur la page d'intro des formulaires single page.

Ma proposition :

skelz0r · 2024-09-24T08:45:30Z

Mon commentaire globale:

Je vous invite à introduire un provider_name sur la définition (qui est en fait le service_name en fait, juste je pense que le naming n'est pas bon, ça se discute), et à ne manipuler qu'exclusivement kind et provider_name pour customiser les wordings de la page d'intro, et de supprimer purement et simplement les références à HubEE dans les vues/helpers/locales.

Pour plus de contexte : #433 (comment) et #433 (comment)

Samuelfaure · 2024-09-25T09:50:42Z

@SchweisguthN @evaspae Wordings generiques pour les providers de service (Hubee) - le but étant de ne pas s'embêter avec le pluriel / les noms de chaque formulaire

unauthenticated page:

skelz0r

Je request change pour des petits détails et le refactor du provider_name (en fait le modèle existe déjà comme souligné dans un commentaire, donc pas besoin d'ajouter la clé qui est déjà en fait renseignée), globalement c'est OK.

config/authorization_definitions.yml

config/editors.yml

app/views/authorization_request_forms/new.html.erb

app/views/authorization_requests/new/default.html.erb

skelz0r · 2024-09-25T12:39:47Z

app/models/authorization_definition.rb

 :provider,
+ :provider_name,


En fait je me rends compte que provider_name est déjà implémenté 😅

Donc dans le code en fait il suffit de remplacer authorization_definition.provider_name en authorization_definition.provider.name partout où c'est utilisé.

On y a pensé sauf que pour cert_dc et Dila le provider name utilisé est "HubEE" or la on veut afficher "CertDC" ou juste "DILA".
C'était pour cette raison qu'en premier lieu, je n'avais rajouté un champs service_name que sur hubee_cert_dc et hubee_dila car j'avais besoin de faire cette distinction uniquement pour ce cas la.

J'avais mis HubEE parce que je pensais que c'était HubEE sur le moment, faut juste changer du coup (parce que le fournisseur de données n'est en effet pas HubEE, c'est juste un moyen (pour faire le parallèle, Rails est le moyen utilisé pour API Entreprise))

Et dtf en l'état on a provider.name qui est HubEE et provider_name qui est CERTDC ce n'est dtf pas logique

Ça ne changera pas le fait que je devrais à un moment rajouter un champs dans authorization_definition car j'aurais besoin d'afficher HubEE a un moment dans l'entête des formulaires.
Je pensais pouvoir réutiliser le provier.name pour cet affichage.

Ça ne changera pas le fait que je devrais à un moment rajouter un champs dans authorization_definition car j'aurais besoin d'afficher HubEE a un moment dans l'entête des formulaires.

T'as des exemples ? Ce n'est pas possible de mettre ça dans le nom de la définition ou du formulaire ?

Il y a un figma et cette demande fait partie du Ticket.

Après plus j'y reflechis et plus ça me dérange de mettre en provider DILA et CertDC, déjà car pour CertDC, le provider serait plutôt le Ministère de la santé et de la prévention / DGS et surtout parceque nous délivrons des habilitions pour l'obtention d'un abonnement HubEE qui permet d'accéder à ces 2 services.

Et avant d'aller plus loin, j'aimerais que les visuels / wording soient validés par @SchweisguthN et @evaspae parce qu'on a pris beaucoup de liberté sur le texte par rapport au texte original.

C'est ok pour moi : on a certes pris de la liberté, mais au nom de la factorisation, ce qui est un argument qui s'entend. Je suis juste pas fan de "aux services du fournisseur DILA" sur la page non-authent, mais j'ai pas de meilleure idée et c'est un détail, donc go.

Un peu plus de contexte ici: https://mattermost.incubateur.net/betagouv/pl/3s9ccm81f7b5tn3gzga9mhyqky

Le tl;dr est surtout que la page change d'objectif entre les maquettes pour HubEE et ce qu'elle est à l'origine (i.e. une intro au formulaire d'habilitation).

config/locales/fr.yml

Add helper in method to display introduction text Refacto using plural from i18n add a service type to authorization_definition.yml Fix an i18n link Add Plural logic to unauthenticated_start page too Update Brakeman Rename service_type into service_name

Modifications wording pluriel services/APIs add provider name to authorization_request_definitions simplify logic in form views simplify logic in form views Change Dinum into DINUM

Fix type on CertDc

- add link and jpg logo

skelz0r mentioned this pull request Sep 20, 2024

Enhance unauthenticated start page according to authorization definition type #437

Merged

Isalafont force-pushed the hubee/DAT-574/correctif-wording branch 2 times, most recently from 01e0bfa to 050e9ec Compare September 23, 2024 09:10

Isalafont self-assigned this Sep 23, 2024

Isalafont force-pushed the hubee/DAT-574/correctif-wording branch from 1393308 to b3c9ae5 Compare September 23, 2024 14:06

Isalafont requested review from skelz0r, JeSuisUnCaillou and Samuelfaure and removed request for skelz0r September 23, 2024 14:08

JeSuisUnCaillou reviewed Sep 23, 2024

View reviewed changes

app/models/authorization_request_form.rb Outdated Show resolved Hide resolved

app/views/authorization_request_forms/new.html.erb Outdated Show resolved Hide resolved

config/authorization_definitions.yml Outdated Show resolved Hide resolved

Samuelfaure removed request for skelz0r and Samuelfaure September 23, 2024 15:03

Isalafont requested a review from Samuelfaure September 23, 2024 15:03

Isalafont force-pushed the hubee/DAT-574/correctif-wording branch from e66bf72 to 7d7fa7f Compare September 23, 2024 15:12

skelz0r reviewed Sep 23, 2024

View reviewed changes

app/models/authorization_definition.rb Outdated Show resolved Hide resolved

Samuelfaure force-pushed the hubee/DAT-574/correctif-wording branch 3 times, most recently from cce5594 to 27414fa Compare September 25, 2024 08:51

Samuelfaure force-pushed the hubee/DAT-574/correctif-wording branch from 4abb6e8 to 48d92cb Compare September 25, 2024 09:55

Isalafont requested a review from skelz0r September 25, 2024 12:25

skelz0r requested changes Sep 25, 2024

View reviewed changes

skelz0r reviewed Sep 25, 2024

View reviewed changes

config/locales/fr.yml Outdated Show resolved Hide resolved

Isalafont added 5 commits September 26, 2024 17:50

Add link to email

f33f89f

Update Brakeman

5fa8770

Modifications wording pluriel services/APIs add provider name to authorization_request_definitions simplify logic in form views simplify logic in form views Change Dinum into DINUM

Align div with app/views/authorization_request_forms/new.html.erb page

b363374

scout commit

41bbf6c

Isalafont force-pushed the hubee/DAT-574/correctif-wording branch from f199ee7 to 98a66da Compare September 26, 2024 16:04

Isalafont changed the title ~~Hubee/dat 574/correctif wording~~ Draft: Hubee/dat 574/correctif wording Sep 26, 2024

Isalafont force-pushed the hubee/DAT-574/correctif-wording branch from 98a66da to 9bee012 Compare September 26, 2024 16:16

Isalafont added 2 commits September 26, 2024 18:23

Revert change to fit to design after meeting

d61d6c7

Fix type on CertDc

Update Brakeman

8fb4874

Isalafont force-pushed the hubee/DAT-574/correctif-wording branch from d057656 to 8fb4874 Compare September 26, 2024 16:24

skelz0r temporarily deployed to sandbox September 26, 2024 17:04 — with GitHub Actions Inactive

Isalafont added 4 commits September 30, 2024 10:32

Fix cucumber test according to wording changes

d904a29

Remove provider_name in code

203d7c5

Add DGS as a CertDC provider instead of Hubee.

f1b8c05

Add DILA as data_providers

5c604be

- add link and jpg logo

skelz0r approved these changes Sep 30, 2024

View reviewed changes

Isalafont temporarily deployed to sandbox September 30, 2024 14:58 — with GitHub Actions Inactive

Isalafont requested a review from SchweisguthN October 1, 2024 07:34

SchweisguthN merged commit 9c76b6d into develop Oct 1, 2024
17 checks passed

SchweisguthN deleted the hubee/DAT-574/correctif-wording branch October 1, 2024 09:11

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Draft: Hubee/dat 574/correctif wording #433

Draft: Hubee/dat 574/correctif wording #433

Isalafont commented Sep 19, 2024 •

edited

Loading

linear bot commented Sep 19, 2024

skelz0r commented Sep 19, 2024

skelz0r commented Sep 19, 2024

skelz0r commented Sep 20, 2024

Isalafont commented Sep 23, 2024

evaspae commented Sep 24, 2024

skelz0r commented Sep 24, 2024

Samuelfaure commented Sep 25, 2024 •

edited

Loading

skelz0r left a comment

skelz0r Sep 25, 2024 •

edited by Isalafont

Loading

Isalafont Sep 25, 2024

skelz0r Sep 25, 2024

skelz0r Sep 25, 2024

Isalafont Sep 25, 2024 •

edited

Loading

skelz0r Sep 25, 2024

Isalafont Sep 25, 2024

SchweisguthN Sep 26, 2024

skelz0r Sep 26, 2024

Draft: Hubee/dat 574/correctif wording #433

Draft: Hubee/dat 574/correctif wording #433

Conversation

Isalafont commented Sep 19, 2024 • edited Loading

linear bot commented Sep 19, 2024

skelz0r commented Sep 19, 2024

skelz0r commented Sep 19, 2024

skelz0r commented Sep 20, 2024

Isalafont commented Sep 23, 2024

evaspae commented Sep 24, 2024

skelz0r commented Sep 24, 2024

Samuelfaure commented Sep 25, 2024 • edited Loading

skelz0r left a comment

Choose a reason for hiding this comment

skelz0r Sep 25, 2024 • edited by Isalafont Loading

Choose a reason for hiding this comment

Isalafont Sep 25, 2024

Choose a reason for hiding this comment

skelz0r Sep 25, 2024

Choose a reason for hiding this comment

skelz0r Sep 25, 2024

Choose a reason for hiding this comment

Isalafont Sep 25, 2024 • edited Loading

Choose a reason for hiding this comment

skelz0r Sep 25, 2024

Choose a reason for hiding this comment

Isalafont Sep 25, 2024

Choose a reason for hiding this comment

SchweisguthN Sep 26, 2024

Choose a reason for hiding this comment

skelz0r Sep 26, 2024

Choose a reason for hiding this comment

Isalafont commented Sep 19, 2024 •

edited

Loading

Samuelfaure commented Sep 25, 2024 •

edited

Loading

skelz0r Sep 25, 2024 •

edited by Isalafont

Loading

Isalafont Sep 25, 2024 •

edited

Loading