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

Ajouter le champ libre “description et méthodologie de calcul” aux indicateurs prédéfinis #3313

Conversation

elisfainstein
Copy link
Contributor

@elisfainstein elisfainstein commented Sep 23, 2024

Ticket

Méthodologie :

L'objectif final est de recâbler le code de façon à ce que le champ "description et méthode de calcul" présent dans les indicateurs perso écrive non plus dans description de la table indicateur_definition, mais dans commentaire de la table indicateur_collectivite.

On fait le changement progressivement en suivant le pattern expand and contract (doc).

Cette PR correspond à la 2e étape du pattern.

Capture d’écran 2024-09-30 à 11 55 03

Fonctionnement :

  • indicateur perso existant : le champ "description et méthode de calcul"

    • écrit à la fois dans description de la table indicateur_definition et dans commentaire de la table indicateur_collectivite ;
    • lit à partir de description de la table indicateur_definition ;
  • indicateur perso nouvellement créé : idem ;

  • indicateur pré-défini : le champ

    • écrit dans commentaire de la table indicateur_collectivite. Il n'écrit pas dans description de la table indicateur_definition car cette colonne abrite déjà des descriptions pré-écrites, qui s'affichent dans le petit id'informations dans l'UI.
    • lit à partir de commentaire de la table indicateur_collectivite.

marc-rutkowski and others added 28 commits October 10, 2024 10:03
…"Ajouter les actions classées <statut>" est activée lors du changement de statut de l'action
… si l'utilisateur n'est pas rattaché à celle-ci
…un utilisateur lors de la transformation du panier en plan d'action
…er mon inscription" pour les utilisateurs non encore rattachés à une collectivité
…les utilisateurs non encore rattachés à une collectivité
* fix: change branch name processing to always be lowercase and tolerate a longer name with hyphens

* fix: fix typo

* fix: change max name length

* fix: remove hyphens in branch name

* fix: limit branch name to 22 characters

* fix: convert - to _ and limit name length to 14 char

* fix: fix typo

* fix: remove - to _ conversion

* doc: add koyeb validation rule explanation
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.

6 participants