- 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
left one minor comment
- π¦πΊ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