Projet

Général

Profil

Bug #54003

avoir un template pour récuperer les données du connecteur Impôts Particulier

Ajouté par Serghei Mihai il y a presque 3 ans. Mis à jour il y a presque 3 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
17 mai 2021
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

De #48668#note-24 :

Le connecteur dans Passerelle renverra une page HTML au navigateur, affichant que tout s'est bien passé, et fait un postMessage sur window.parent avec les données obtenues.

Le template, appliqué à un champ w.c.s doit contenir le JS qui:

  • intercepte le postMessage
  • pose l' id des données dans le champ caché
  • affiche un message expliquant ce qui s'est passé
  • ferme l'onglet/l'iframe

Prévoir un bouton pour virer les données du champ et permet de recommencer.


Fichiers

0001-add-template-for-FranceConnect-connector-item-field-.patch (6,55 ko) 0001-add-template-for-FranceConnect-connector-item-field-.patch Benjamin Dauvergne, 27 mai 2021 11:11
Firefox_Screenshot_2021-05-27T09-14-36.278Z.png (14,6 ko) Firefox_Screenshot_2021-05-27T09-14-36.278Z.png Benjamin Dauvergne, 27 mai 2021 11:15
Firefox_Screenshot_2021-05-27T09-14-15.217Z.png (26,2 ko) Firefox_Screenshot_2021-05-27T09-14-15.217Z.png Benjamin Dauvergne, 27 mai 2021 11:15
Test connecteur FC.png (24,1 ko) Test connecteur FC.png Serghei Mihai, 27 mai 2021 11:19
Firefox_Screenshot_2021-05-27T09-51-54.664Z.png (12,6 ko) Firefox_Screenshot_2021-05-27T09-51-54.664Z.png Benjamin Dauvergne, 27 mai 2021 11:52
0001-templates-add-template-for-FC-information-retrieval-.patch (12,3 ko) 0001-templates-add-template-for-FC-information-retrieval-.patch Serghei Mihai, 04 juin 2021 11:40
Test connecteur FC.png (371 ko) Test connecteur FC.png Serghei Mihai, 04 juin 2021 11:41
0001-templates-add-template-for-FC-information-retrieval-.patch (12,4 ko) 0001-templates-add-template-for-FC-information-retrieval-.patch Serghei Mihai, 04 juin 2021 12:11
0001-templates-add-template-for-FC-information-retrieval-.patch (12,7 ko) 0001-templates-add-template-for-FC-information-retrieval-.patch Serghei Mihai, 07 juin 2021 17:59
0001-templates-add-template-for-FC-information-retrieval-.patch (12,7 ko) 0001-templates-add-template-for-FC-information-retrieval-.patch Serghei Mihai, 07 juin 2021 18:05
0001-templates-add-template-for-FC-information-retrieval-.patch (13,9 ko) 0001-templates-add-template-for-FC-information-retrieval-.patch Serghei Mihai, 23 juin 2021 11:11
Firefox_Screenshot_2021-06-23T09-23-49.789Z.png (8,67 ko) Firefox_Screenshot_2021-06-23T09-23-49.789Z.png Benjamin Dauvergne, 23 juin 2021 11:24
Firefox_Screenshot_2021-06-23T09-38-40.645Z.png (14 ko) Firefox_Screenshot_2021-06-23T09-38-40.645Z.png Benjamin Dauvergne, 23 juin 2021 11:39
Firefox_Screenshot_2021-06-23T09-38-54.940Z.png (13,8 ko) Firefox_Screenshot_2021-06-23T09-38-54.940Z.png Benjamin Dauvergne, 23 juin 2021 11:40

Demandes liées

Lié à Passerelle - Development #53879: connecteur FranceConnect / API Impôts ParticulierFermé10 mai 2021

Actions

Révisions associées

Révision 4b1586f5 (diff)
Ajouté par Serghei Mihai il y a presque 3 ans

templates: add template for FC information retrieval (#54003)

Historique

#2

Mis à jour par Serghei Mihai il y a presque 3 ans

#3

Mis à jour par Serghei Mihai il y a presque 3 ans

  • Description mis à jour (diff)
#4

Mis à jour par Benjamin Dauvergne il y a presque 3 ans

  • Assigné à mis à Benjamin Dauvergne
#5

Mis à jour par Benjamin Dauvergne il y a presque 3 ans

Première proposition, je vais avoir besoin d'aide :
  • pour savoir dans quel fichier je dois déposer les bouts de style (et éventuellement les renommer pour coller à la politique de nommage des classes)
  • pour revoir le style pour obtenir un rendu moins pourri et plus proche de celui du bouton FranceConnect natif tout en conservant le coté un peu libre du texte (on ne veut pas "S'identifier avec FranceConnect" comme partout, puisque le but c'est quand mêmes d'obtenir les données des impôts à la fin, pas juste valider son identité avec FC)
#6

Mis à jour par Benjamin Dauvergne il y a presque 3 ans

Pour référence le rendu de mon bouton tout moisi et le rendu natif avec le thème FC (tout le bouton est un SVG les caractères ne viennent pas d'une police mais ont été aplatis dans le SVG, je ne sais pas si on peut trouver une police proche).

#7

Mis à jour par Serghei Mihai il y a presque 3 ans

J'avais commencé à bosser dessus.

En terme de vocabulaire je voyais plutôt "Obtenir avec France Connect".

#9

Mis à jour par Benjamin Dauvergne il y a presque 3 ans

  • Assigné à changé de Benjamin Dauvergne à Serghei Mihai

Je passe la main à Serghei et Thomas.

#10

Mis à jour par Benjamin Dauvergne il y a presque 3 ans

Petit changement par rapport à mon interface entre le connecteur et le template, en cas d'erreur ça fait window.opener.postMessage({"error": "message technique", "data": {..détail de l'erreur...}} au niveau du template il ne faut pas afficher ça tel quel, juste dire, Les services de l'état sont en panne. mais ce serait bien de logger ça dans la console js par contre, que ce soit facile à analyser.

#11

Mis à jour par Serghei Mihai il y a presque 3 ans

Voici une première version avec interception de l'erreur et affichage du message indiquant qu'il y a eu une erreur.

#13

Mis à jour par Benjamin Dauvergne il y a presque 3 ans

À quoi sert ce bout de style ?

<style>
  .select2-container {
      display: none;
  }
</style>

class="franceconnect--error" style="display: none" 

Je ne connais pas la politique à ce sujet mais je penser que tu peux poser ça dans le .scss aussi.


Le code d'ouverture de popup ne marche pas, les options sont à passer sous forme de chaîne1, pas d'arguments nommés à la python, au passage reprendre le centrage minimal et les options de présentation qui sont coté passerelle pour l'ouverture de la popup.

+                var left = (screen.width/2)-(w/2);
+                var top = (screen.height/2)-(h/2);
+                return window.open(
+                    url, title,
+                    'toolbar=no, location=no, directories=no, status=no, menubar=no, scrollbars=yes, resizable=no, copyhistory=no, width='+w+', height='+h+', top='+top+', left='+left);

1 https://developer.mozilla.org/fr/docs/Web/API/Window/open

#14

Mis à jour par Serghei Mihai il y a presque 3 ans

  • Statut changé de Solution proposée à En cours

Benjamin Dauvergne a écrit :

À quoi sert ce bout de style ?

Cacher le champ select2.


[...]

Je ne connais pas la politique à ce sujet mais je penser que tu peux poser ça dans le .scss aussi.

Je l'ai mis inline pour qu'en backoffice, lors du preview du formulaire, le message ne soit pas affiché.


Le code d'ouverture de popup ne marche pas, les options sont à passer sous forme de chaîne1, pas d'arguments nommés à la python, au passage reprendre le centrage minimal et les options de présentation qui sont coté passerelle pour l'ouverture de la popup.

[...]

Ok, je vais corriger ça.

#15

Mis à jour par Benjamin Dauvergne il y a presque 3 ans

Serghei Mihai a écrit :

Cacher le champ select2.

Ok, je n'avais pas vu le {{ block.super }} vu que ça n'était pas dans mon template d'origine; y a-t-il a des chose utiles pour nous dans le template select_jsonp.html ? On pourrait peut-être hériter directement de qommon/forms/widget.html à la place.

#16

Mis à jour par Benjamin Dauvergne il y a presque 3 ans

Benjamin Dauvergne a écrit :

Serghei Mihai a écrit :

Cacher le champ select2.

Ok, je n'avais pas vu le {{ block.super }} vu que ça n'était pas dans mon template d'origine; y a-t-il a des chose utiles pour nous dans le template select_jsonp.html ? On pourrait peut-être hériter directement de qommon/forms/widget.html à la place.

Je me plussoie tout seul, vu qu'on est quand même loin de l'usage select/JSONP je serais plutôt pour que le template n'utilise pas du tout l'héritage dans le bloc widget-control, ça sera plus simple pour celui qui viendra relire plus tard (pas besoin de se demander s'il existe une interaction entre les deux ou pas) et aucune chaîne liée à select2 ne doit apparaître non plus; la seule chose à conserver c'est la gestion de data-initial-display-value, tout le reste est à gérer avec des <input type="hidden"/>, la valeur raw et la valeur display, la valeur display peut-être visible mais en read-only, il me semble qu'il faut qu'elle soit soumise et donc qu'elle ait un attribut "name" pour ça fonctionne correctement, cf. extrait de qommon.form.js :

      // create an additional hidden field to hold the label of the selected
      // option, it is necessary as the server may not have any knowledge of
      // possible options.
      var $input_display_value = $('<input>', {
              type: 'hidden',
              name: $(elem).attr('name') + '_display',
              value: $(elem).data('initial-display-value')
      });
#17

Mis à jour par Serghei Mihai il y a presque 3 ans

C'est plus simple, en effet.

#20

Mis à jour par Benjamin Dauvergne il y a presque 3 ans

Ok testé en remplaçant template-franceconnect par template-franceconnect-data ça roule, mais par contre le champ d'affichage est un peu étroit par défaut.

#21

Mis à jour par Serghei Mihai il y a presque 3 ans

Souci de cache CSS? Le champ a une largeur de 100%.

#22

Mis à jour par Benjamin Dauvergne il y a presque 3 ans

Souci de make css surtout, ça va mieux. Reste :

  • la police du bouton détonne un peu niveau taille et ressort mal sur le fond bleu (c'est un peu floud comme un effet blur): mais c'est très subjectif, si personne d'autre n'a la même réaction que moi tu peux ignorer,
  • le champ est trop près du bord droit :
#23

Mis à jour par Serghei Mihai il y a presque 3 ans

Benjamin Dauvergne a écrit :

  • le champ est trop près du bord droit :

Tu as ça sur quel thème? J'ai essayé sur plusieurs en local (genre Clapotis-les-canards) et le champ ne dépasse pas la largeur du widget.

#24

Mis à jour par Benjamin Dauvergne il y a presque 3 ans

Thème CNIL, je vérifie sur Clapotis.

#25

Mis à jour par Benjamin Dauvergne il y a presque 3 ans

C'est ok sur Clapotis il y a un peu de padding à droite, go.

#26

Mis à jour par Serghei Mihai il y a presque 3 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 4b1586f5fd16f9564966e44e58ee61d79386f126 (origin/main)
Author: Serghei Mihai <smihai@entrouvert.com>
Date:   Fri May 21 11:10:31 2021 +0200

    templates: add template for FC information retrieval (#54003)
#27

Mis à jour par Frédéric Péters il y a presque 3 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF