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

rfc: POC pour diviser le composant EditBouquet #145

Closed
wants to merge 3 commits into from

Conversation

edelagnier
Copy link
Contributor

@edelagnier edelagnier commented Nov 8, 2023

Superseded by #223

RFC

Depends on #162
Related to issue #144

Context or situation

Le composant editBouquet devient trop gros pour être gérable sur le moyen terme.

Problem encountered by users

Invisible pour l'usager, mais pénible pour les dévs.

Proposal of how to solve the problem

Décomposer en plusieurs composants et sous-composants tout en gardant l'accès aux informations.

What changes

Dans la page d'acceuil s'affiche une version brute du formulaire de création/édition de bouquet.

En éditant les premiers champs, les informations sont automatiquement retranscrite dans l'étape 4 récapitulative.

En cliquant sur ajouter la donnée dans l'étape 3 :

  • un nouveau dataset est ajouté dans la section "Données déjà sélectionnées" de l'étape 3.
  • le dataset est aussi automatiquement reporté dans l'étape 4 récapitulative

Breaking changes

La preuve de concept est sommairement affichée dans la page d'acceuil

Technical changes

  • Preuve de concept pour juger de la complexité du chantier.
  • Découpage en plusieurs composant, prise en compte de cas particulier étape 3 (composant contenant lui même un formulaire)
  • Utilisation de typescript dans les sous-composants. Pas d'impact sur le reste du code et permets de garantir le format des données afin de prévenir un comportement imprévu à l'utilisation

@@ -0,0 +1,68 @@
<template>
Copy link
Contributor

Choose a reason for hiding this comment

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

Pourrait-on appeler ce composant DatasetProperties ? C'est le nom métier qu'on lui a donnée.

Copy link
Contributor Author

@edelagnier edelagnier Nov 8, 2023

Choose a reason for hiding this comment

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

Pas de souci, je n'étais de toute façon pas satisfaite de ce nom là. On aurait donc l'interface typescript DatasetProperties et le composant DatasetPropertiesForm ?
Je voulais aussi vous poser la question quand à l'usage de Bouquet vs Topic (ce que j'ai compris être les bouquets dans l'API datagouv ?)

@streino streino marked this pull request as draft November 9, 2023 10:00
@streino streino requested review from YeLnatSs and streino November 9, 2023 10:00
@streino streino changed the title [Do not merge] POC pour diviser le composant EditBouquet POC pour diviser le composant EditBouquet Nov 9, 2023
Base automatically changed from step2-add-chantier-and-thematique to main November 9, 2023 10:41
@bonjourmauko bonjourmauko added the question Question ou discussion requise sur le sujet label Nov 16, 2023
@bonjourmauko bonjourmauko added this to the V1 : Découvrabilité par cas d’usage milestone Nov 16, 2023
@bonjourmauko bonjourmauko changed the title POC pour diviser le composant EditBouquet rfc: POC pour diviser le composant EditBouquet Nov 23, 2023
@bonjourmauko
Copy link
Contributor

Superseded by #223

@bonjourmauko bonjourmauko deleted the spike/divide_editBouquetCmpt branch December 6, 2023 11:40
@bonjourmauko bonjourmauko removed the question Question ou discussion requise sur le sujet label Dec 6, 2023
@streino streino added the question Question ou discussion requise sur le sujet label Feb 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Question ou discussion requise sur le sujet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants