search

lab-64 / split-the-bill-server

Backend for the Split the Bill App
4 stars 0 forks source link

188: Ensure consistency on bill update #189

Closed GR0ZA closed 3 months ago

GR0ZA commented 3 months ago

Ensure consistency on bill update via comparing updatedAt from the updateDTO with updatedAt from the database to check whether an update happened in the meantime.

Closes #188 Closes #191

f-odc commented 3 months ago

I see you implement an optimistic locking procedure. We have to discuss whether this will be the best approach. Rejecting updates if another update is performed earlier could cause a very bad user experience. I will explain an alternative approach in the related issue. We will talk and discuss what approach suits better our use case.

f-odc commented 3 months ago

Work on this PR after PR #194 is merged.

Important: