Skip to content

K
kohinos-tav
Opened by  

add parameter for paying entity & update helper text for frequency of reconversion

{{ resolvedDiscussionCount }}/{{ discussionCount }} {{ resolvedCountText }} resolved
  • Damien Moulard @DamienM

    changed target branch from sprint-2-montpellier to develop

    changed target branch from sprint-2-montpellier to develop

    Toggle commit list
  • Yvon Kerdoncuff
    @Yvon started a discussion on the diff
    Resolved by Yvon Kerdoncuff
    fixtures/tavtest/fixtures-tav.yaml
    791 791 description: "Numéro de téléphone à afficher dans le formulaire de contact si défini"
    792 792 value: ''
    793 793 mandatory: 1
    794 gp31:
    • Yvon Kerdoncuff @Yvon commented
      Master

      La modification du paramètre dans ces fichiers de fixtures n'est pas indispensable (c'est éventuellement un confort d'avoir ces paramètres préremplis quand on setup une instance de test en local pour nous pour avoir déjà des users créés etc., par contre en pratique quand je fais un staging pour un client ssa je n'utilise jamais les fixtures de test mais directement les versions tavpro/ssagirondepro).

      La modification des fichiers de fixtures est par contre non suffisante. Pour que le nouveau paramètre de configuration soit effectivement ajouté à la base de données, il est nécessaire d'écrire un fichier de migration. Voir par exemple le fichier Version20240321101500.php pour l'ajout des paramètres RIB gestionnaire pour reconversion.

      Edited by Yvon Kerdoncuff
    • Damien Moulard @DamienM commented
      Master

      Je n'ai pas voulu créer de migration pour pas que toutes les instances aient une valeur par défaut à cette variable, ce qui n'aurait aucun sens (ici j'ai juste complété les fixtures de test), et je considérait qu'on créerait la variable manuellement dans l'onglet configuration. Mais effectivement on peut créer une migration avec le champ 'value' à null comme dans la migration que tu indiques, c'est bien plus pérenne.

    • Damien Moulard @DamienM commented
      Master

      rajouté

    • Yvon Kerdoncuff @Yvon commented
      Master

      Ah OK je savais pas qu'on pouvait ajouter depuis l'interface. Bien vu de l'initialiser à null.

    Please register or sign in to reply
  • Yvon Kerdoncuff
    @Yvon started a discussion on an old version of the diff
    Last updated by Yvon Kerdoncuff
    src/Form/Type/PrestataireInfosFormType.php
    95 95 if ($this->container->getParameter('tav_env') && $this->container->getParameter('automatisation_reconversion')) {
    96 96
    97 97 $mlcName = $this->em->getRepository(GlobalParameter::class)->val(GlobalParameter::MLC_NAME_SMALL);
    98 $helpMsqg = "Fréquence à laquelle je souhaite que la caisse commune de l'alimentation (via l'association Acclimat'action)"
    98
    99 // If a paying entity is defined, add to to help message
    100 $payingEntity = $this->em->getRepository(GlobalParameter::class)->val(GlobalParameter::PAYING_ENTITY_NAME);
    101 $payingEntityMsg = ($payingEntity !== null && $payingEntity !== '') ? "(via l'association {$payingEntity})" : "";
    • Yvon Kerdoncuff @Yvon commented
      Master

      Vu le nom du paramètre qui invite à indiquer n'importe quel type de structure, il est peut-être un peu dommage de faire l'hypothèse qu'il s'agit d'une association. J'écrirais simplement (via {$payingEntity}).

    • Damien Moulard @DamienM commented
      Master

      Hum en soit je suis d'accord mais j'ai peur que le message ne soit pas très clair si on ne connaît pas l'organisme en question.

    • Damien Moulard @DamienM commented
      Master

      Bah tient, on pourrait remplacer "association" par "organisme" ?

    • Yvon Kerdoncuff @Yvon

      changed this line in version 4 of the diff

      changed this line in version 4 of the diff

      Toggle commit list
    • Yvon Kerdoncuff @Yvon commented
      Master

      J'ai fait la modif.

    Please register or sign in to reply
  • Damien Moulard @DamienM

    added 1 commit

    • 51285cf5 - add migration to add global parameter

    Compare with previous version

    added 1 commit

    • 51285cf5 - add migration to add global parameter

    Compare with previous version

    Toggle commit list
  • Yvon Kerdoncuff @Yvon

    added 2 commits

    • 6adea703 - via l'association -> via l'organisme
    • 57406761 - Merge branch '7376-reconversion-frquency-entity-text' of…

    Compare with previous version

    added 2 commits

    • 6adea703 - via l'association -> via l'organisme
    • 57406761 - Merge branch '7376-reconversion-frquency-entity-text' of…

    Compare with previous version

    Toggle commit list
  • Yvon Kerdoncuff @Yvon

    merged

    merged

    Toggle commit list
  • Yvon Kerdoncuff @Yvon

    mentioned in commit 05c661fc

    mentioned in commit 05c661fc

    Toggle commit list
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment