Skip to content

feat(list): implement optimistic locking for ShoppingList and ListItem#93

Closed
juliogitdev wants to merge 11 commits intoomatheusmesmo:developfrom
juliogitdev:fix/optimistic-locking-shared-lists
Closed

feat(list): implement optimistic locking for ShoppingList and ListItem#93
juliogitdev wants to merge 11 commits intoomatheusmesmo:developfrom
juliogitdev:fix/optimistic-locking-shared-lists

Conversation

@juliogitdev
Copy link
Copy Markdown

Implemented Optimistic Locking for ShoppingList and ListItem entities
to prevent data loss in concurrent edit scenarios (last-write-wins problem).

Changes

  • Added @Version annotation to ShoppingList and ListItem entities
  • Added migration V4__add_optimistic_locking.sql
  • Added handler for ObjectOptimisticLockingFailureException returning HTTP 409

Closes #21

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 2, 2026

🤖 Hi @juliogitdev, I've received your request, and I'm working on it now! You can track my progress in the logs for more details.

Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖🎉 Thank you for your contribution! Your pull request has been submitted successfully. A maintainer from the team will review it as soon as possible. We appreciate your support in making this project better!

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 2, 2026

🤖 I'm sorry @juliogitdev, but I was unable to process your request. Please see the logs for more details.

@matheusandre1
Copy link
Copy Markdown
Collaborator

LGTM

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 4, 2026

🤖 Hi @juliogitdev, I've received your request, and I'm working on it now! You can track my progress in the logs for more details.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 4, 2026

🤖 I'm sorry @juliogitdev, but I was unable to process your request. Please see the logs for more details.

Copy link
Copy Markdown
Owner

@omatheusmesmo omatheusmesmo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @juliogitdev !

Great job—thanks for your contribution! Since this project isn’t in production, feel free to simply adjust the existing migration scripts rather than creating a new one.

@omatheusmesmo
Copy link
Copy Markdown
Owner

Hello @juliogitdev ! A better approach would be update your remote repository and then run the following commands inside your local feature branch.

git fetch origin
git rebase origin main

@omatheusmesmo
Copy link
Copy Markdown
Owner

Hey @juliogitdev , is there any update on this?

@omatheusmesmo omatheusmesmo deleted the branch omatheusmesmo:develop April 18, 2026 15:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants