entrouvert
/
lingo
15
0
Fork 0
Code Issues Pull Requests 7 Releases Wiki Activity

misc: use hobo.multitenant.spooler (#76747) #55

Open
bdauvergne wants to merge 1 commits from wip/76747-utiliser-hobo-multitenant-spoole into main
pull from: wip/76747-utiliser-hobo-multitenant-spoole
merge into: entrouvert:main
Conversation 0 Commits 1 Files Changed 6 +57 -92
bdauvergne commented 2023-04-18 13:43:18 +02:00
Owner
Copy Link

La version "spooler" de populate_from_draft faisait plus de vérification que la version standard du modèle, j'ai conservé ces vérifications, qui sont là je suppose pour pallier au fait qu'on est plus dans le même temps/même transaction qu'au moment de l'appel à Pool.promote() mais ça mériterait quelques commentaires d'explications sur ce qu'on redoute/de quoi on se protège via ces vérifications.

PS: commentaires que j'ajouterai si on me les précise.

La version "spooler" de populate_from_draft faisait plus de vérification que la version standard du modèle, j'ai conservé ces vérifications, qui sont là je suppose pour pallier au fait qu'on est plus dans le même temps/même transaction qu'au moment de l'appel à Pool.promote() mais ça mériterait quelques commentaires d'explications sur ce qu'on redoute/de quoi on se protège via ces vérifications. PS: commentaires que j'ajouterai si on me les précise.
bdauvergne force-pushed wip/76747-utiliser-hobo-multitenant-spoole from 4c700db596 to 8518c2ea68 2023-04-18 13:52:11 +02:00 Compare
All checks were successful
gitea/lingo/pipeline/head This commit looks good
Details
This pull request has changes conflicting with the target branch.
  • debian/control
  • lingo/invoicing/models.py
  • lingo/settings.py
You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.
git checkout -b wip/76747-utiliser-hobo-multitenant-spoole main
git pull origin wip/76747-utiliser-hobo-multitenant-spoole

Step 2:

Merge the changes and update on Gitea.
git checkout main
git merge --no-ff wip/76747-utiliser-hobo-multitenant-spoole
git push origin main
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
1 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/lingo#55
No description provided.
Delete Branch "wip/76747-utiliser-hobo-multitenant-spoole"

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?