entrouvert/wcs
entrouvert
/
wcs
14
1
Fork 0
Code Issues Pull Requests 34 Releases Activity

create new draft if current draft has been removed (#75021) #977

Merged
fpeters merged 1 commits from wip/75021-ignore-nothing-to-update into main 2024-01-15 11:57:38 +01:00
Conversation 0 Commits 1 Files Changed 2 +40
fpeters commented 2023-12-29 19:40:17 +01:00
Owner
Copy Link

Pas bien sûr de la situation ici; à remonter à l'origine du NothingToUpdate c'est #65748 et dans la version initiale du patch c'était prévu pour concerner uniquement l'autosave (pour ne pas écraser un brouillon par une version d'une page précédente qui aurait déclenché l'autosave) mais lors de la relecture ça a été élargi à couvrir toutes les situations où un brouillon est enregistré et je n'ai pas l'impression que ça ait été mesuré.

Dans cette PR j'ai d'abord tenté d'imaginer la situation et comment la rattraper mais comme ça arrive très rarement (il y a une entrée dans sentry il y a trois mois, ce ticket a 10 mois), il y a trop peu d'éléments et finalement je limite à l'affichage d'un message d'erreur à l'usager.

Pas bien sûr de la situation ici; à remonter à l'origine du NothingToUpdate c'est #65748 et dans la version initiale du patch c'était prévu pour concerner uniquement l'autosave (pour ne pas écraser un brouillon par une version d'une page précédente qui aurait déclenché l'autosave) mais lors de la relecture ça a été élargi à couvrir toutes les situations où un brouillon est enregistré et je n'ai pas l'impression que ça ait été mesuré. Dans cette PR j'ai d'abord tenté d'imaginer la situation et comment la rattraper mais comme ça arrive très rarement (il y a une entrée dans sentry il y a trois mois, ce ticket a 10 mois), il y a trop peu d'éléments et finalement je limite à l'affichage d'un message d'erreur à l'usager.
fpeters force-pushed wip/75021-ignore-nothing-to-update from 092d4d367f to 155573a748 2023-12-30 13:38:29 +01:00 Compare
fpeters force-pushed wip/75021-ignore-nothing-to-update from 155573a748 to 13210a3794 2024-01-13 12:27:04 +01:00 Compare
fpeters changed title from wip: create new draft if current draft has been removed (#75021) to create new draft if current draft has been removed (#75021) 2024-01-13 14:28:21 +01:00
lguerin approved these changes 2024-01-15 11:53:36 +01:00
fpeters merged commit caee075927 into main 2024-01-15 11:57:38 +01:00
fpeters deleted branch wip/75021-ignore-nothing-to-update 2024-01-15 11:57:38 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
lguerin
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/wcs#977
No description provided.
Delete Branch "wip/75021-ignore-nothing-to-update"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?