// this up() migration is auto-generated, please modify it to your needs
23
$this->addSql("INSERT INTO global_parameter (id, name, description, value, mandatory) VALUES (UUID(), 'ACTIVATE_ADHERENTS_BALANCE_CEILING', 'Active ou non les fonctionnalités liées au plafond de solde d\'un adhérent', null, '1')");
Autant pour la story #7544, je trouvais que ce n'est pas si grave que ça parce que c'était que du wording, mais constate que tu as suivi la même logique ici où c'est plus problématique.
De façon générale, quand on fait un développement pour un autre client, il ne faut pas que ça casse ce qui a été fait pour les clients pour qui nous avons développé une fonctionnalité précédemment. Or, si on met à jour ce code chez ssa-G, comme nous n'avons pas de process nous permettant de nous souvenir ou d'automatiser toutes les reconfigurations nécessaires, nous allons l'oublier.
Donc ils n'auront plus le bouton pour appliquer le plafond. Du coup, ils ne vont pas comprendre, ils vont chercher pourquoi, puis ils vont nous faire un mail pour nous demander pourquoi, et on devra leur répondre. Bref, ce n'est pas souhaitable, surtout qu'il existe des alternatives.
Une proposition de résolution est de mettre la valeur par défaut à true dans la migration.
Pour la story #7544, même si ce n'est plus le sujet ici parce que je l'ai validée, l'idéal serait de mettre un texte par défaut également, soit dans la migration, soit dans le code.
Autant pour la story #7544, je trouvais que ce n'est pas si grave que ça parce que c'était que du wording, mais constate que tu as suivi la même logique ici où c'est plus problématique.
De façon générale, quand on fait un développement pour un autre client, il ne faut pas que ça casse ce qui a été fait pour les clients pour qui nous avons développé une fonctionnalité précédemment. Or, si on met à jour ce code chez ssa-G, comme nous n'avons pas de process nous permettant de nous souvenir ou d'automatiser toutes les reconfigurations nécessaires, nous allons l'oublier.
Donc ils n'auront plus le bouton pour appliquer le plafond. Du coup, ils ne vont pas comprendre, ils vont chercher pourquoi, puis ils vont nous faire un mail pour nous demander pourquoi, et on devra leur répondre. Bref, ce n'est pas souhaitable, surtout qu'il existe des alternatives.
Une proposition de résolution est de mettre la valeur par défaut à true dans la migration.
Pour la story #7544, même si ce n'est plus le sujet ici parce que je l'ai validée, l'idéal serait de mettre un texte par défaut également, soit dans la migration, soit dans le code.
Même réponse que sur télégram, de plus le paramètre requiert l'activation à minima de household_based_allowance (on devrait peut-être le rajouter en description d'ailleurs ?), donc il ne me semble pas judicieux de le mettre à true par défaut (je pense à la fusion avec kohinos par exemple).
Même réponse que sur télégram, de plus le paramètre requiert l'activation à minima de household_based_allowance (on devrait peut-être le rajouter en description d'ailleurs ?), donc il ne me semble pas judicieux de le mettre à true par défaut (je pense à la fusion avec kohinos par exemple).
changed this line in [version 2 of the diff](https://gl.cooperatic.fr/agplv3/kohinos-tav/merge_requests/130/diffs?diff_id=3832&start_sha=12da036da753313446a07c6e9b71fbd91c1e7553#d52370176b7666e5a6668124147965636e2f5aa7_23_23)
added 31 commits
<ul><li>12da036d...9796cd68 - 29 commits from branch <code>sprint-2-montpellier</code></li><li>f7facefb - Merge branch 'sprint-2-montpellier' into 7500-balance-ceiling-parameter</li><li>651f09e2 - set default value and improve description for global parameter activate_adherents_balance_ceiling</li></ul>
[Compare with previous version](https://gl.cooperatic.fr/agplv3/kohinos-tav/merge_requests/130/diffs?diff_id=3832&start_sha=12da036da753313446a07c6e9b71fbd91c1e7553)
added 1 commit
<ul><li>d63179f6 - set default value for reconversion_frequency_help_text to value initialy used in…</li></ul>
[Compare with previous version](https://gl.cooperatic.fr/agplv3/kohinos-tav/merge_requests/130/diffs?diff_id=3833&start_sha=651f09e2eabd8d6afa0879e7eb043df1a22e422b)
