- Issue created by @tedbow
- 🇺🇸United States tedbow Ithaca, NY, USA
Needs review for the general idea. Then we could clean it up or just scrap it
- 🇧🇪Belgium wim leers Ghent 🇧🇪🇪🇺
This screams "race conditions", and it's definitely in the Auto-save realm, so asking @larowlan to review the general idea :)
- 🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10
I think this is a good idea, we can update the cached version in the store
- 🇧🇪Belgium wim leers Ghent 🇧🇪🇪🇺
Given @larowlan's +1, let's clarify the plan/intent here. I'd like the issue title to reflect what this MR aims to do, not describe the symptom 🙏
- 🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10
Added a related issue 🐛 Publishing the currently edit page causes 'another user has changed this content' error Active which is in the same ballpark
- 🇧🇪Belgium wim leers Ghent 🇧🇪🇪🇺
Merge this on the client side with the existing list of auto-saves
Does that client-side issue already exist?
Can we get the title updated to reflect what the scope is?
NW per @larowlan's https://git.drupalcode.org/project/experience_builder/-/merge_requests/9...
- 🇺🇸United States tedbow Ithaca, NY, USA
I am investigating whether this should be postponed on 📌 [PP-1] Enforce conflict enforcement outside of tests and e2e tests Postponed
- 🇺🇸United States tedbow Ithaca, NY, USA
We should have an explanation in the summary of the back-end/front-end interaction that leads to this problem
- Status changed to Postponed: needs info
18 days ago 12:20pm 26 June 2025 - 🇺🇸United States tedbow Ithaca, NY, USA
@wim leers thanks. Sorry I wasn't clear. I updated summary
I don't think we can solve this on the back-end allow and depending on how the front-end whats to handle this we might not need any back-end changes
I proposed what I consider the "safest" solution but I am not sure that will offer the UX we want
- 🇫🇮Finland lauriii Finland
I think we can release without fixing this bug. We should still prioritize this as soon as possible but it doesn't seem like a must have for the releases.