Run deletions in transactions #467
Labels
No labels
A: API
A: Backend
A: Federation
A: Front-End
A: I18N
A: Meta
A: Security
Build
C: Bug
C: Discussion
C: Enhancement
C: Feature
Compatibility
Dependency
Design
Documentation
Good first issue
Help welcome
Mobile
Rendering
S: Blocked
S: Duplicate
S: Incomplete
S: Instance specific
S: Invalid
S: Needs Voting/Discussion
S: Ready for review
Suggestion
S: Voted on Loomio
S: Wontfix
No milestone
No project
No assignees
2 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: Plume/Plume#467
Loading…
Add table
Reference in a new issue
No description provided.
Delete branch "%!s()"
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?
So that if something fails, all the previous deletions are also cancelled. I'm thinking of blog/user deletion here, for which deletion cascade have been "manually" implemented.
Plume version (and/or commit): 0.2.0
It's not the only thing that we should run in transactions, for instance creating a new user is a two step process, filling some fields, then updating others. It should be done in a transaction to avoid both race conditions (create A success, create B fail, update A success), and prevent a user from having certain fields ever empty