Skip to content

ETQ usager, je peux personnaliser la liste des dossiers (3/3 - valeurs sur les cartes)#13384

Open
kara22 wants to merge 14 commits into
feat/dossiers-personnalisation-pr2from
feat/dossiers-personnalisation-pr3
Open

ETQ usager, je peux personnaliser la liste des dossiers (3/3 - valeurs sur les cartes)#13384
kara22 wants to merge 14 commits into
feat/dossiers-personnalisation-pr2from
feat/dossiers-personnalisation-pr3

Conversation

@kara22

@kara22 kara22 commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@kara22
kara22 force-pushed the feat/dossiers-personnalisation-pr2 branch from 008cc25 to a8948b1 Compare June 30, 2026 16:14
@kara22
kara22 force-pushed the feat/dossiers-personnalisation-pr3 branch from cff073a to a2d2715 Compare June 30, 2026 16:15
@kara22
kara22 force-pushed the feat/dossiers-personnalisation-pr2 branch from a8948b1 to 60b2ccd Compare June 30, 2026 16:19
@kara22
kara22 force-pushed the feat/dossiers-personnalisation-pr3 branch from a2d2715 to 3bed8c2 Compare June 30, 2026 16:19
@kara22
kara22 force-pushed the feat/dossiers-personnalisation-pr2 branch from 60b2ccd to d21bb25 Compare July 6, 2026 14:23
@kara22
kara22 force-pushed the feat/dossiers-personnalisation-pr3 branch from 3bed8c2 to 0852fc0 Compare July 6, 2026 14:23

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

quitte à l'extraire ce serait plutôt dans app/services

@kara22
kara22 force-pushed the feat/dossiers-personnalisation-pr3 branch from 8e21ab8 to 67ea9c2 Compare July 20, 2026 11:59
@kara22
kara22 force-pushed the feat/dossiers-personnalisation-pr2 branch from d21bb25 to 397afb9 Compare July 20, 2026 11:59

@colinux colinux left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Je fais une review globale des PR en plus des différents poins inline mais certains sujets concernent les autres PR :

  • il y a un combobox vide lorsqu'il n'y a pas de champ sélectionnable
  • style des sections (je t'ai pingé sur mattermost pour ça)
  • le champ commune n'affiche quel son code postal (car c'est la première colonne, le nom standardisé est ailleurs)
  • la nouvelle table bloque la suppression des procedures car il' n'y a pas de dependant destroy sur Procedure
  • l'index génère bcp de queries (à cause de personnalisation_available?, la récupération de chaque revisions...). Peut-être qu'on pourra optimiser que dans un 2nd temps
  • personnalisation_available? compte current_user.dossiers.visible_by_user alors que la liste affichée inclut aussi les dossiers reçus en invitation.
  • j'ai du vérifier à la main, ce serait bien d'un avoir test qui couvre ce cas : j'ai personnalisé sur un champ , qui est supprimé à la réision suivante => pas d'erreur à l'affichage des dossiers ni à la liste des personnalisations

Un autre problème à cause du filter des dynamic_type une colonne enregistrée en base (personnaliastion) mais qui n'est plus disponible provoquerait une erreur 500 (on change parfois de noms de colonnes etc…). Je sais pas trop comment gérer ça, peut-être ismplement un rescue autour, ou ne plus pas passer par Column.find ici mais réutiliser personnalisable_columns

Et je regrette le mot français personnalisation et dérivé au milieu de noms anglais dans le code !!! mais c'est trop tard

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Je trouve que le emptyHint vient polluer ce select alors que c'est très spécifique à cas d'usage. Il aura mieux eu sa place hors du composant react, mais j'imagine que c'est pas trop possible car comme tout se fait en react et pas par le serveur il faut le rendre dedans ?

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.

2 participants