Skip to content

Admin: Amélioration d'une description de champ [GEN-2325]#6075

Merged
tonial merged 1 commit into
masterfrom
alaurent/boost
May 5, 2025
Merged

Admin: Amélioration d'une description de champ [GEN-2325]#6075
tonial merged 1 commit into
masterfrom
alaurent/boost

Conversation

@tonial

@tonial tonial commented May 5, 2025

Copy link
Copy Markdown
Contributor

🤔 Pourquoi ?

Indiquez le problème que nous sommes en train de résoudre et les objectifs métiers ou techniques qui sont visés par ces changements.

🍰 Comment ?

Décrivez en quelques mots la solution retenue et mise en oeuvre, les difficultés ou problèmes rencontrés. Attirez l'attention sur les décisions d'architecture ou de conception importantes.

🚨 À vérifier

  • Mettre à jour le CHANGELOG_breaking_changes.md ?
  • Ajouter l'étiquette « Bug » ?

🏝️ Comment tester ?

Les instructions pour reproduire le problème, les profils de test, le parcours spécifique à utiliser, etc. Si vous disposez d'une recette jetable, mettre l'URL pour tester dans cette partie.

💻 Captures d'écran

@tonial tonial added the modifié Modifié dans le changelog. label May 5, 2025
@tonial tonial requested review from leo-naeka and vincentporte May 5, 2025 07:43
@tonial tonial self-assigned this May 5, 2025
@notion-workspace

Copy link
Copy Markdown

@leo-naeka leo-naeka left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Ok pour moi.

Mais on aurait aussi pu revoir le wording sur l'abstract pour éviter de surcharger la demande non ?
Par exemple : "prescripteur habilité lié à cette prolongation"

Ça semble être la seule FK possible vers un PH

@tonial

tonial commented May 5, 2025

Copy link
Copy Markdown
Contributor Author

Le problème c'est que ça peut être soit une prolongation, soit une demande de prolongation, donc la formulation générique que tu proposes ne marche pas tout à fait.

@tonial

tonial commented May 5, 2025

Copy link
Copy Markdown
Contributor Author

@leo-naeka sinon, je le change proprement dans le modèle en mettant validated_by dans chaque modèle avec le bon verbose name. (cf commit WIP qui suit)

@leo-naeka

Copy link
Copy Markdown
Contributor

Ça semble être la seule FK possible vers un PH

My bad, il y a processed_by en plus sur la demande.

la formulation générique que tu proposes ne marche pas tout à fait

En partant du principe qu'il n'y avait qu'un seul PH lié à ces deux modèles je pensais que si, mais effectivement non :/


C'est une histoire de labels, j'avoue avoir une préférence pour surcharger dans l'init du coup, on préserve un fonctionnement commun en dédupliquant et on prend pas le risque de louper l'un des deux à l'avenir 🤷‍♂️
Désolé pour le double taf…

@tonial tonial force-pushed the alaurent/boost branch from 7b15d17 to d533d6f Compare May 5, 2025 14:25
@tonial tonial enabled auto-merge May 5, 2025 14:26
@tonial tonial added this pull request to the merge queue May 5, 2025
Merged via the queue into master with commit 0dba263 May 5, 2025
@tonial tonial deleted the alaurent/boost branch May 5, 2025 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

modifié Modifié dans le changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants