Skip to content

Improve column mappings#162

Open
raph-topo wants to merge 1 commit intobetagouv:mainfrom
raph-topo:column-mappings
Open

Improve column mappings#162
raph-topo wants to merge 1 commit intobetagouv:mainfrom
raph-topo:column-mappings

Conversation

@raph-topo
Copy link

Text & types

Copy link
Collaborator

@fflorent fflorent left a comment

Choose a reason for hiding this comment

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

Thank you!

Waiting for @CamilleLegeron approval too before merging.

title: "adresse (source)",
type: "Any",
title: "Adresse (source)",
type: "Text",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Il me semble que si cette colonne est d'un autre type pour X raison alors on ne pourra pas la sélectionner parce que là on exige que ce soit de type Text, et donc dans la liste des colonnes selectionnable on ne verra que celle de type Text.
J'avais volontairement laissé type "any" pour ne pas limiter l'usage du widget et ne pas avoir d'effet de bord, si c'est une volonté de faire comme ça, ok pour moi mais bien tester les usages et ce pour toutes les colonnes

Copy link
Collaborator

Choose a reason for hiding this comment

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

Merci pour ton retour.

J'avais volontairement laissé type "any" pour ne pas limiter l'usage du widget et ne pas avoir d'effet de bord, si c'est une volonté de faire comme ça, ok pour moi mais bien tester les usages et ce pour toutes les colonnes

Qu'entends-tu par là ?

Copy link
Author

Choose a reason for hiding this comment

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

Les colonnes de type Any sont toujours proposées au choix pour les widget, quel qu'en soit le réglage.

Inversement, quand un widget accepte Any, on peut sélectionner tous les types de colonnes (inclus les Reference List qui feraient sauter le widget).

name: "latitude",
title: "Latitude (destination)",
type: "Any",
type: "Numeric",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Idem là, est-ce qu'on décide que le widget ne doit pas pouvoir fonctionner si le type de la colonne Lat est text ?

Copy link
Collaborator

Choose a reason for hiding this comment

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

C'est l'usage avec le widget officiel pour les cartes. On ne peut sélectionner une colonne latitude ou longitude que si elle est de type numérique.

Copy link
Author

Choose a reason for hiding this comment

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

Latitude & longitude sont par nature des nombres, non ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants