Skip to content

  • Projects
  • Groups
  • Snippets
  • Help
  • This project
    • Loading...
  • Sign in / Register
K
kohinos-tav
  • Overview
    • Overview
    • Details
    • Activity
    • Cycle Analytics
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Charts
  • Issues 0
    • Issues 0
    • List
    • Board
    • Labels
    • Milestones
  • Merge Requests 1
    • Merge Requests 1
  • CI / CD
    • CI / CD
    • Pipelines
    • Jobs
    • Schedules
    • Charts
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Members
    • Members
  • Collapse sidebar
  • Activity
  • Graph
  • Charts
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • agplv3
  • kohinos-tav
  • Merge Requests
  • !122

Merged
Opened Jan 07, 2025 by Damien Moulard@DamienM 
  • Report abuse
Report abuse

set a default value to presta reconversion frequency

×

Check out, review, and merge locally

Step 1. Fetch and check out the branch for this merge request

git fetch origin
git checkout -b 7377-presta-reconv-frequency-default origin/7377-presta-reconv-frequency-default

Step 2. Review the changes locally

Step 3. Merge the branch and fix any conflicts that come up

git checkout sprint-2-montpellier
git merge --no-ff 7377-presta-reconv-frequency-default

Step 4. Push the result of the merge to GitLab

git push origin sprint-2-montpellier

Note that pushing to GitLab requires write access to this repository.

Tip: You can also checkout merge requests locally by following these guidelines.

  • Discussion 3
  • Commits 2
  • Changes 2
{{ resolvedDiscussionCount }}/{{ discussionCount }} {{ resolvedCountText }} resolved
  • Yvon Kerdoncuff
    @Yvon started a discussion on the diff Jan 13, 2025
    Last updated by Damien Moulard Jan 17, 2025
    src/Entity/Prestataire.php
    342 342 * Fréquence de reconversion en cas d'automatisation des reconversions.
    • Yvon Kerdoncuff @Yvon commented Jan 13, 2025
      Master

      Les modifications que tu proposes sont effectivement nécessaires pour renseigner la valeur pour les prestataires qui seront ajoutées dans le futur, mais elles ne sont pas suffisantes pour renseigner la valeur dans les prestataires qui n'ont actuellement pas de mode de reconversion choisi.

      Pour plus de détails, sur les effets de chacune des deux modifications que tu proposes et une recommandation de commande à passer dans un fichier de migration pour basculer les prestataires actuellement sans fréquence de reconversion : https://chatgpt.com/share/6784da76-4094-8004-924c-7c89e2a10bc4.

      Les modifications que tu proposes sont effectivement nécessaires pour renseigner la valeur pour les prestataires qui seront ajoutées dans le futur, mais elles ne sont pas suffisantes pour renseigner la valeur dans les prestataires qui n'ont actuellement pas de mode de reconversion choisi. Pour plus de détails, sur les effets de chacune des deux modifications que tu proposes et une recommandation de commande à passer dans un fichier de migration pour basculer les prestataires actuellement sans fréquence de reconversion : https://chatgpt.com/share/6784da76-4094-8004-924c-7c89e2a10bc4.
    • Damien Moulard @DamienM commented Jan 17, 2025
      Master

      Effectivement, je n'avais pas pensé aux prestataires existants. Je rajoute une ligne à la migration. J'allais partir sur rajouter une commande à appeler manuellement pour pouvoir choisir la valeur, mais on met déjà la valeur "once_a_month" comme valeur par défaut pour les nouvelles entités, donc autant appliquer cette valeur directement et donc ne pas s'embêter avec une commande.

      Effectivement, je n'avais pas pensé aux prestataires existants. Je rajoute une ligne à la migration. J'allais partir sur rajouter une commande à appeler manuellement pour pouvoir choisir la valeur, mais on met déjà la valeur "once_a_month" comme valeur par défaut pour les nouvelles entités, donc autant appliquer cette valeur directement et donc ne pas s'embêter avec une commande.
    • Damien Moulard @DamienM commented Jan 17, 2025
      Master

      fait

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

    added 1 commit

    • 9ebe690c - add default value to existing presta reconversionFrequency in migration

    Compare with previous version

    Jan 17, 2025

    added 1 commit

    • 9ebe690c - add default value to existing presta reconversionFrequency in migration

    Compare with previous version

    added 1 commit <ul><li>9ebe690c - add default value to existing presta reconversionFrequency in migration</li></ul> [Compare with previous version](https://gl.cooperatic.fr/cooperatic/kohinos-tav/merge_requests/122/diffs?diff_id=3718&start_sha=845c70034bb28cce747c87a8c7cf055a263affae)
    Toggle commit list
  • Yvon Kerdoncuff @Yvon

    merged

    Jan 17, 2025

    merged

    merged
    Toggle commit list
  • Yvon Kerdoncuff @Yvon

    mentioned in commit 588199a5

    Jan 17, 2025

    mentioned in commit 588199a5

    mentioned in commit 588199a5b9bef813f107649c53efa3b7a6e54ea1
    Toggle commit list
  • Write
  • Preview
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 sign in to comment
Assignee
No assignee
Assign to
None
Milestone
None
Assign milestone
Time tracking
Reference: agplv3/kohinos-tav!122
×

Revert this merge request

This will create a new commit in order to revert the existing changes.

Switch branch
Cancel
A new branch will be created in your fork and a new merge request will be started.
×

Cherry-pick this merge request

Switch branch
Cancel
A new branch will be created in your fork and a new merge request will be started.