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

wip/84895-scrib-abstract-soap #428

Merged
tnoel merged 2 commits from wip/84895-scrib-abstract-soap into main 2023-12-22 12:19:36 +01:00
Conversation 1 Commits 2 Files Changed 4 +104 -6
tnoel commented 2023-12-19 12:08:05 +01:00
Owner
Copy Link
No description provided.
tnoel added 2 commits 2023-12-19 12:08:05 +01:00
gitea/passerelle/pipeline/head There was a failure building this commit Details
9abc34029f scrib: use abstract SOAP model (#84895)
tnoel force-pushed wip/84895-scrib-abstract-soap from 9abc34029f to beced20123 2023-12-19 12:12:26 +01:00 Compare
bdauvergne reviewed 2023-12-19 12:41:28 +01:00
passerelle/apps/soap/models.py Outdated
@ -250,0 +251,4 @@
class SOAPConnector(AbstractSOAPConnector):
pass
bdauvergne commented 2023-12-19 12:41:28 +01:00
Owner
Copy Link

Model.Meta n'est pas hérité je pense, il faut redéfinir verbose_name ici (et changer celui plus haut en "Abstract trucmuche").

Model.Meta n'est pas hérité je pense, il faut redéfinir verbose_name ici (et changer celui plus haut en "Abstract trucmuche").
tnoel commented 2023-12-19 17:44:17 +01:00
Author
Owner
Copy Link

C'est pris en compte, merci !

C'est pris en compte, merci !
tnoel marked this conversation as resolved
tnoel force-pushed wip/84895-scrib-abstract-soap from beced20123 to 8387acd3f0 2023-12-19 16:55:19 +01:00 Compare
tnoel force-pushed wip/84895-scrib-abstract-soap from 8387acd3f0 to 21a14db63c 2023-12-19 17:41:18 +01:00 Compare
tnoel force-pushed wip/84895-scrib-abstract-soap from 21a14db63c to 36f27abdea 2023-12-19 17:43:45 +01:00 Compare
tnoel force-pushed wip/84895-scrib-abstract-soap from 36f27abdea to 31a0828d6d 2023-12-22 11:07:39 +01:00 Compare
tnoel commented 2023-12-22 11:42:01 +01:00
Author
Owner
Copy Link

Testé «en réel» avec un migrate_schemas, tout se passe «correctement». Si un connecteur nantes-scrib était instancié, il faut juste supprimer le connecteur «soap» qui s'est retrouvé créé par héritage. Je ne préfère pas jouer cela dans les migrations, parce que c'est un peu compliqué mais aussi inutile : nulle part on n'a d'instance en place.

Testé «en réel» avec un migrate_schemas, tout se passe «correctement». Si un connecteur nantes-scrib était instancié, il faut juste supprimer le connecteur «soap» qui s'est retrouvé créé par héritage. Je ne préfère pas jouer cela dans les migrations, parce que c'est un peu compliqué mais aussi inutile : nulle part on n'a d'instance en place.
tnoel changed title from WIP: wip/84895-scrib-abstract-soap to wip/84895-scrib-abstract-soap 2023-12-22 11:42:06 +01:00
bdauvergne approved these changes 2023-12-22 12:16:54 +01:00
tnoel merged commit 31a0828d6d into main 2023-12-22 12:19:36 +01:00
tnoel deleted branch wip/84895-scrib-abstract-soap 2023-12-22 12:19:36 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
bdauvergne
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 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#428
No description provided.
Delete Branch "wip/84895-scrib-abstract-soap"

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?