entrouvert
/
passerelle
15
0
Fork 0
Code Issues Pull Requests 16 Releases Activity

soap: fix initial migration (#84761) #424

Closed
tnoel wants to merge 1 commits from wip/84761-soap-fix-initial-migration into main
pull from: wip/84761-soap-fix-initial-migration
merge into: entrouvert:main
Conversation 2 Commits 1 Files Changed 1 +2 -2
tnoel commented 2023-12-14 17:37:52 +01:00
Owner
Copy Link

Je n'ai pas créé de nouvelle migration car je pense que ça n'a pas d'impact côté SQL (?)

Je n'ai pas créé de nouvelle migration car je pense que ça n'a pas d'impact côté SQL (?)
tnoel added 1 commit 2023-12-14 17:37:52 +01:00
gitea/passerelle/pipeline/head This commit looks good Details
754658458f soap: fix initial migration (#84761)
nroche approved these changes 2023-12-14 17:47:46 +01:00
nroche left a comment
Owner
Copy Link

(connecteur utilisé nul part en prod à ma connaissance)

(connecteur utilisé nul part en prod à ma connaissance)
tnoel commented 2023-12-14 17:49:32 +01:00
Author
Owner
Copy Link

(jabber)

(ça ne sera pas poussé, mais ça m'aidera à m'assurer que j'ai pas erroré)

Ca ne sera pas poussé ce soir, je précise :)

> (jabber) > > (ça ne sera pas poussé, mais ça m'aidera à m'assurer que j'ai pas erroré) Ca ne sera pas poussé _ce soir_, je précise :)
tnoel closed this pull request 2023-12-14 17:49:32 +01:00
tnoel deleted branch wip/84761-soap-fix-initial-migration 2023-12-14 17:55:18 +01:00
bdauvergne commented 2023-12-14 19:13:13 +01:00
Owner
Copy Link

Je n'ai pas créé de nouvelle migration car je pense que ça n'a pas d'impact côté SQL (?)

Ça n'a d'impact sur rien a priori, à part une éventuelle migration de donnée qui utiliserait le modèle virtuel et se référerait dans ce cas à la valeur qui est dans la migration. Dans le code de passerelle c'est la valeur du modèle déclaré qui prévaut. Ça vaudrait le coup d'avoir un script check-migrations dans passerelle.

> Je n'ai pas créé de nouvelle migration car je pense que ça n'a pas d'impact côté SQL (?) Ça n'a d'impact sur rien a priori, à part une éventuelle migration de donnée qui utiliserait le modèle virtuel et se référerait dans ce cas à la valeur qui est dans la migration. Dans le code de passerelle c'est la valeur du modèle déclaré qui prévaut. Ça vaudrait le coup d'avoir un script check-migrations dans passerelle.
All checks were successful
gitea/passerelle/pipeline/head This commit looks good
Details

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No reviewers
nroche
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/passerelle#424
No description provided.
Delete Branch "wip/84761-soap-fix-initial-migration"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?