Begin of the insertion#### Context

In the context of changes made to polling in ##54733, the error message concerning conflicting modifications to a work package is particularly misleading. This is an effort to improve that.

#### Steps to reproduce

In the current production version of OpenProject (Community):

* User A enters edit mode of a work package (any field, not necessarily description)

* User A makes changes

* User B modifies the same work package in parallel

* User A tries to save the (unbenowst to them) now-stale work package


####End of the insertion **Buggy behaviour**

Begin of the insertion* The user receives the following error, with a link to reload the page to load the updated version.

* However, if the user is still editing an attribute (eg. the description is in edit mode), the userEnd of the insertion Begin of the deletion (ParimalEnd of the deletion will Begin of the insertionstill have the stale content for that attribute.

* The conflict is then not resolved.


<img class="image_resized op-uc-image op-uc-image_inline" style="width:663px;" src="/api/v3/attachments/234553/content">End of the insertion Begin of the deletion complete)End of the deletion

Begin of the insertion####End of the insertion **Expected behaviour**

* Begin of the insertionWhen a change is detected in the work package by another user or in another window (i.e., the work package is now &quot;stale&quot;):End of the insertion Begin of the deletion **Warning** (automatically appears)End of the deletion

* Begin of the insertionAutomatically display a **warning message**.

* End of the insertion Use the existing Primerised Refresh banner, but in a warning yellow. Begin of the insertion

End of the insertion Begin of the deletion

End of the deletion * Text: &quot;This page has been updated by someone else. To not lose your edits, copy them locally and reload to view the updated version.&quot; Begin of the insertion

End of the insertion Begin of the deletion

End of the deletion * Action: **Discard changes and reload**. Begin of the insertion

End of the insertion Begin of the deletion (Browser refresh)

End of the deletion * Begin of the insertionThis triggers a browser refresh.

End of the insertion Begin of the deletion **Error:**

End of the deletion * Begin of the insertionIfEnd of the insertion Begin of the deletion SupressEnd of the deletion the Begin of the insertionuser, despiteEnd of the insertion Begin of the deletion warning banner in favour ofEnd of the deletion the Begin of the insertionwarning, tries to save their changes:

End of the insertion Begin of the deletion error one.

End of the deletion * Begin of the insertionDisplay an **error message**.End of the insertion Begin of the deletion RedEnd of the deletion

* Use Begin of the insertionthe existingEnd of the insertion Begin of the deletion aEnd of the deletion Primerised Begin of the insertionRefresh banner, in red. (IfEnd of the insertion Begin of the deletion banner component (ifEnd of the deletion possible, else use the Begin of the insertionexisting AngularEnd of the insertion Begin of the deletion currentEnd of the deletion one) Begin of the insertion

End of the insertion Begin of the deletion

End of the deletion * Begin of the insertionSupress the warning banner in favour of the error one.

* End of the insertion Text: &quot;Could not save your changes because of conflicting modifications. To not lose your edits, copy them locally and reload to view the updated version.&quot; Begin of the insertion

End of the insertion Begin of the deletion

End of the deletion * Action: **Discard changes and reload** (Browser refresh)&#39; Begin of the insertion


#### Reference

A reference of the Primerised Refresh banner:

<img class="image_resized op-uc-image op-uc-image_inline" style="width:843px;" src="/api/v3/attachments/234555/content">

####End of the insertion Begin of the deletion


End of the deletion **Notes**

* Known limitation: Some edits cannot really be copied, so the message doesn&#39;t work in all cases. Begin of the insertionIt is nevertheless an improvement.End of the insertion

* This is a stop-gap solution to slightly improve on what already exists. Begin of the insertion

End of the insertion Begin of the deletion (Describe what happens now and why it&#39;s not good)

End of the deletion * We&#39;ll work on a better solution as a separate feature, Begin of the insertionperhaps:

* End of the insertion Begin of the deletion perhapsEnd of the deletion with Begin of the insertiondiff-ing that allows users to better understand the conflict

* concurrent editing of _different_ attributes doesn&#39;t trigger the error


### Environment information

**Community**
OpenProject 14.6

**Browser:**
Firefox, Mac (desktop)

**Language**
ENEnd of the insertion Begin of the deletion diff-ing.End of the deletion