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
  • !24

Merged
Opened Jan 04, 2023 by Yvon Kerdoncuff@Yvon 
  • Report abuse
Report abuse

4286 annulation modifications 4176

  • Discussion 5
  • Commits 6
  • Changes 3
{{ resolvedDiscussionCount }}/{{ discussionCount }} {{ resolvedCountText }} resolved
  • Damien Moulard @DamienM commented Jan 05, 2023
    Master

    Attention, il ne faut jamais supprimer une migration ! Sinon tous les autres environnements auront des problèmes.

    Une meilleure pratique est de remodifier les modèles et ensuite de regénérer une migration qui fera les opérations dans l'autre sens, ou créer manuellement une migration qui fait les opérations dans l'autre sens.

    Edited Jan 05, 2023 by Damien Moulard
    Attention, il ne faut jamais supprimer une migration ! Sinon tous les autres environnements auront des problèmes. Une meilleure pratique est de remodifier les modèles et ensuite de regénérer une migration qui fera les opérations dans l'autre sens, ou créer manuellement une migration qui fait les opérations dans l'autre sens.
  • Yvon Kerdoncuff @Yvon commented Jan 05, 2023
    Master

    Je ne maîtrise pas exactement les conséquences de cette suppression même dans ce cas particulier mais j'avoue que ça sent pas bon.

    Le cas présent est un peu particulier, pour deux raisons :

    1. c'est une migration qui contient seulement des data et aucune modification sur le modèle de données
    2. les modifications impliquées par la migration n'ont aucune importance puisque le ROLE additionnel attribué n'est plus utilisé dans le code

    Bon, je remets la migration ça c'est sûr. Je suis tenté de vider la fonction UP. Une migration vide, c'est pas très propre, mais un peu plus qu'une migration qui sert à rien ?

    Edited Jan 05, 2023 by Yvon Kerdoncuff
    Je ne maîtrise pas exactement les conséquences de cette suppression même dans ce cas particulier mais j'avoue que ça sent pas bon. Le cas présent est un peu particulier, pour deux raisons : 1. c'est une migration qui contient seulement des data et aucune modification sur le modèle de données 2. les modifications impliquées par la migration n'ont aucune importance puisque le ROLE additionnel attribué n'est plus utilisé dans le code Bon, je remets la migration ça c'est sûr. Je suis tenté de vider la fonction UP. Une migration vide, c'est pas très propre, mais un peu plus qu'une migration qui sert à rien ?
  • Yvon Kerdoncuff @Yvon

    added 1 commit

    • 6e7524d6 - a migration should not be deleted

    Compare with previous version

    Jan 05, 2023

    added 1 commit

    • 6e7524d6 - a migration should not be deleted

    Compare with previous version

    added 1 commit <ul><li>6e7524d6 - a migration should not be deleted</li></ul> [Compare with previous version](https://gl.cooperatic.fr/cooperatic/kohinos-tav/merge_requests/24/diffs?diff_id=1991&start_sha=96c898d0caf6cac62822d8e2b87ed9af94f1ffc2)
    Toggle commit list
  • Yvon Kerdoncuff @Yvon commented Jan 05, 2023
    Master

    J'ai push une proposition avec migration vidée de son contenu.

    J'ai push une proposition avec migration vidée de son contenu.
  • Damien Moulard @DamienM commented Jan 05, 2023
    Master

    Alors, toujours pas. Si la migration a été faite en prod ou sur un autre environnement, les changements ont été apportés en base. Donc, si on supprime ou vide la migration, les environnements qui n'ont pas fait la migration ne seront pas dans le même état que ceux qui l'ont faite.

    Si les modifications n'ont aucune importance, le mieux est donc sans doute de laisser la migration telle quelle.

    Alors, toujours pas. Si la migration a été faite en prod ou sur un autre environnement, les changements ont été apportés en base. Donc, si on supprime ou vide la migration, les environnements qui n'ont pas fait la migration ne seront pas dans le même état que ceux qui l'ont faite. Si les modifications n'ont aucune importance, le mieux est donc sans doute de laisser la migration telle quelle.
  • Yvon Kerdoncuff @Yvon

    added 1 commit

    • 8e1b51c6 - once migration has been merged, do not change it at all

    Compare with previous version

    Jan 06, 2023

    added 1 commit

    • 8e1b51c6 - once migration has been merged, do not change it at all

    Compare with previous version

    added 1 commit <ul><li>8e1b51c6 - once migration has been merged, do not change it at all</li></ul> [Compare with previous version](https://gl.cooperatic.fr/cooperatic/kohinos-tav/merge_requests/24/diffs?diff_id=1996&start_sha=6e7524d666eb2257fd35373dee90b3e58d830b9f)
    Toggle commit list
  • Yvon Kerdoncuff @Yvon commented Jan 06, 2023
    Master

    Et voilà.

    Et voilà.
  • Damien Moulard @DamienM

    merged

    Jan 09, 2023

    merged

    merged
    Toggle commit list
  • Damien Moulard @DamienM

    mentioned in commit 0a7b3ecd

    Jan 09, 2023

    mentioned in commit 0a7b3ecd

    mentioned in commit 0a7b3ecd14b15f5ef1dbc4b51a1c9ed676799337
    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!24
×

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.