Feat: regroupement de facteurs #583

narduin · 2024-11-14T20:49:43Z

Ajout de la notion de regroupement group dans les bouquets.

Jusqu'ici:

Ajout du composant Vue 3 Multiselect pour la sélection/création de regroupements.

Test de ce même composant pour la sélection des datasets (quelques bugs en cours).

Fix ecolabdata/ecospheres#145
Fix ecolabdata/ecospheres#491
Fix ecolabdata/ecospheres#518

create disclosure component add group field to datasetEditModal start group logic implementation

netlify · 2024-11-14T20:50:37Z

✅ Deploy Preview for ecospheres ready!

Name	Link
🔨 Latest commit	`0beeb17`
🔍 Latest deploy log	https://app.netlify.com/sites/ecospheres/deploys/676599d9cc11ca0007218401
😎 Deploy Preview	https://deploy-preview-583--ecospheres.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

netlify · 2024-11-14T20:51:21Z

✅ Deploy Preview for meteo-france ready!

Name	Link
🔨 Latest commit	`0beeb17`
🔍 Latest deploy log	https://app.netlify.com/sites/meteo-france/deploys/676599d9cae31b000844d74c
😎 Deploy Preview	https://deploy-preview-583--meteo-france.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

abulte

Cool, ça prend déjà forme 😎

package.json

src/components.d.ts

src/components/DisclosureWidget.vue

src/components/bouquets/BouquetDatasetList.vue

src/utils/bouquetGroups.ts

src/components/datasets/DatasetAddToBouquetModal.vue

refactor group selection in datasetAddToBouquetModdal

remove unused dep

sort group alphabetically in composable + styles

small cleanup

abulte

💪 😎

j'ai une erreur en console <path>... au chargement de la page de détails
problème à la suppression du dernier JDD d'un groupe (KO et erreurs console)
les problèmes sur le nouveau select dataset que tu as vues je crois
petite régression sur le style quand pas de JDD

src/components/DisclosureWidget.vue

src/components/forms/dataset/SelectDataset.vue

src/utils/bouquetGroups.ts

narduin · 2024-12-04T11:59:29Z

j'ai une erreur en console <path>... au chargement de la page de détails

Problème d'export svg c'est corrigé !

problème à la suppression du dernier JDD d'un groupe (KO et erreurs console)

Je n'ai pas réussi à reproduire, tu aurais étape par étape ?

abulte

Attendons jusqu'à demain voir si data.gouv.fr a des remarques particulières avant de merger, sinon ok pour moi.

abulte

Le padding sous le filtre couverture territoriale a changé

narduin · 2024-12-17T15:52:10Z

Le padding sous le filtre couverture territoriale a changé

C'est corrigé, j'en ai profité pour mettre le même chevron !

abulte

On perd le cadre de l'input avec une couverture sélectionnée

narduin · 2024-12-17T16:24:42Z

On perd le cadre de l'input avec une couverture sélectionnée

Corrigé aussi. Le texte n'est pas super bien centré à cause d'un margin bottom (de la librairie) j'ai pas surchargé vu qu'on va remplacer le composant bientôt 🤞

DanFluture275 · 2024-12-19T11:52:11Z

Retours recette :

Bloquant :

Boutons modifier et supprimer accessibles quand on n'a pas les droits d'édition sur le bouquet

Mineurs :

Problème couleur dropdown "Sans regroupement" quand édition d'un JDD dans lequel rien n'a été saisi dans regroupement
Pas de msg d'erreur quand on valide un renommage de regroupement avec un champs vide
Pas de limite de caractère à 100 pour le nom du facteur (dans la popin renommage + formulaire création/édition de JDD)

* feat(groups-filter): logic and markup for factor filtering * feat(groups-sommaire): add sommaire logic and styles * feat(groups-filtre): optimize code and fix small bugs moved functions to composables and updated references fixed an anchor link bug inside DsfrTabs with a quick and dirty CSS hack feat(groups-filtre): optimize code and fix small bugs 🧙‍ * feat(groups-filter): fix searchComponent regression

src/components/forms/SelectTopicGroup.vue

DanFluture275

testé PO OK

abulte

😱 ecolabdata/ecospheres#518

narduin · 2024-12-20T15:43:27Z

😱 ecolabdata/ecospheres#518

Toute modification (regroupement ou facteur) faite pendant le filtrage entraîne ce comportement. C'est parce qu'on filtre les datasetsProperties et qu'on réassigne ces même datasetsProperties filtrés au renommage/édition. La version filtrée écrase ainsi la version d'origine.

DanFluture275 · 2024-12-23T14:34:58Z

@narduin J'ai encore un comportement bizarre lors de la modif d'un JDD pendant qu'un filtre est actif.
Dans un bouquet, j'ai :

facteur 1
facteur 2

Je filtre sur "2", j'édite "facteur 2" pour le renommer en "facteur 2 doublon", j'enregistre.
--> "facteur 1" a disparu, et j'ai "facteur 2" + "facteur 2 doublon".

Comportement attendu : avoir "facteur 1" et "facteur 2 doublon"

Vidéo en pj
https://github.com/user-attachments/assets/920c858b-7c75-4cd3-b299-03866e611b0c

narduin added 6 commits November 11, 2024 19:09

add group property to datasetProperties

a644db4

create disclosure component add group field to datasetEditModal start group logic implementation

WIP

53ae317

WIP

bdc4a70

switched to a composable for groups related operations

5457a0e

WIP

30a2cc5

dataset edit and remove refactor to account for groups

0003b97

narduin added 2 commits November 14, 2024 21:56

reset bouquet.ts

165d9c5

add group selection from dataset details form

006f36a

abulte reviewed Nov 18, 2024

View reviewed changes

narduin added 10 commits November 19, 2024 15:56

move datasetIndex to composable

c02dcca

refactor group selection in datasetAddToBouquetModdal

move group selection to component

d622ead

remove unused dep

add rename group feature

42e39f6

cleanup

ae9c485

delete group + move delete dataset to composable

74cbc97

WIP - more utils for groups + groups styles

8290ec0

WIP

1a8785f

refactor disclosure structure + styles

5ce2848

change forms order for group selection

44b33af

add teleport to disclosure modal to test in preview

e908eca

narduin requested a review from abulte November 28, 2024 16:52

fix delete group bug +

e5a56b9

sort group alphabetically in composable + styles

narduin force-pushed the feat/regroupement-facteurs branch from 2339bee to e5a56b9 Compare December 3, 2024 13:31

narduin marked this pull request as ready for review December 3, 2024 13:45

add group to export

2663dbc

small cleanup

narduin force-pushed the feat/regroupement-facteurs branch from 445dce4 to 2663dbc Compare December 3, 2024 13:49

abulte reviewed Dec 4, 2024

View reviewed changes

review fixes and refactor

7082889

Merge branch 'main' into feat/regroupement-facteurs

3b9557c

abulte previously approved these changes Dec 17, 2024

View reviewed changes

abulte requested a review from DanFluture275 December 17, 2024 15:20

abulte reviewed Dec 17, 2024

View reviewed changes

feat(groups): harmonize select styles

9b84316

narduin dismissed abulte’s stale review via 9b84316 December 17, 2024 15:51

abulte reviewed Dec 17, 2024

View reviewed changes

feat(groups): edit multiselect styles

82790b4

abulte previously approved these changes Dec 18, 2024

View reviewed changes

feat(groups): fix permissions + validation

980bfc4

narduin dismissed abulte’s stale review via 980bfc4 December 19, 2024 12:46

feat(groups): fix empty string validation

3a7c351

narduin requested a review from abulte December 19, 2024 12:57

abulte reviewed Dec 19, 2024

View reviewed changes

src/components/forms/SelectTopicGroup.vue Outdated Show resolved Hide resolved

feat(groups): revert empty group attribution

c99129e

narduin force-pushed the feat/regroupement-facteurs branch from 52c24a4 to c99129e Compare December 19, 2024 15:31

DanFluture275 previously approved these changes Dec 19, 2024

View reviewed changes

abulte requested changes Dec 19, 2024

View reviewed changes

feat(groups-filter): fix filtering side effects

0beeb17

narduin dismissed DanFluture275’s stale review via 0beeb17 December 20, 2024 16:22

narduin requested review from abulte and Thesauruv December 20, 2024 16:25

abulte approved these changes Dec 23, 2024

View reviewed changes

narduin requested a review from DanFluture275 December 23, 2024 09:51

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Feat: regroupement de facteurs #583

Feat: regroupement de facteurs #583

narduin commented Nov 14, 2024 •

edited

Loading

netlify bot commented Nov 14, 2024 •

edited

Loading

netlify bot commented Nov 14, 2024 •

edited

Loading

abulte left a comment

abulte left a comment

narduin commented Dec 4, 2024 •

edited

Loading

abulte left a comment •

edited

Loading

abulte left a comment

narduin commented Dec 17, 2024

abulte left a comment

narduin commented Dec 17, 2024

DanFluture275 commented Dec 19, 2024 •

edited by narduin

Loading

DanFluture275 left a comment

abulte left a comment

narduin commented Dec 20, 2024

DanFluture275 commented Dec 23, 2024

Feat: regroupement de facteurs #583

Are you sure you want to change the base?

Feat: regroupement de facteurs #583

Conversation

narduin commented Nov 14, 2024 • edited Loading

netlify bot commented Nov 14, 2024 • edited Loading

✅ Deploy Preview for ecospheres ready!

netlify bot commented Nov 14, 2024 • edited Loading

✅ Deploy Preview for meteo-france ready!

abulte left a comment

Choose a reason for hiding this comment

abulte left a comment

Choose a reason for hiding this comment

narduin commented Dec 4, 2024 • edited Loading

abulte left a comment • edited Loading

Choose a reason for hiding this comment

abulte left a comment

Choose a reason for hiding this comment

narduin commented Dec 17, 2024

abulte left a comment

Choose a reason for hiding this comment

narduin commented Dec 17, 2024

DanFluture275 commented Dec 19, 2024 • edited by narduin Loading

DanFluture275 left a comment

Choose a reason for hiding this comment

abulte left a comment

Choose a reason for hiding this comment

narduin commented Dec 20, 2024

DanFluture275 commented Dec 23, 2024

narduin commented Nov 14, 2024 •

edited

Loading

netlify bot commented Nov 14, 2024 •

edited

Loading

netlify bot commented Nov 14, 2024 •

edited

Loading

narduin commented Dec 4, 2024 •

edited

Loading

abulte left a comment •

edited

Loading

DanFluture275 commented Dec 19, 2024 •

edited by narduin

Loading