zestedesavoir / zds-site

Cœur du projet technique de Zeste de Savoir
https://zestedesavoir.com
Other
268 stars 161 forks source link

Perte de texte avec le système de prévention de l'édition simultanée #6643

Open Arnaud-D opened 2 months ago

Arnaud-D commented 2 months ago

Description du bug

Dans certains cas, la protection contre l'édition simultanée crée plus de problème qu'elle n'en résout.

Comment reproduire ?

Comportement attendu

Il faudrait avoir le choix d'écraser la version distante ou non, pouvoir comparer, et surtout ne pas perdre ce qu'on vient d'écrire.

Informations techniques

Dans la version de dev à l'heure où j'écris ça (https://github.com/zestedesavoir/zds-site/commit/75989e6a27431451effbedb725fb0ff2cbc07f4b).

philippemilink commented 2 months ago

Pour info, le code en question : https://github.com/zestedesavoir/zds-site/blob/75989e6a27431451effbedb725fb0ff2cbc07f4b/zds/tutorialv2/views/contents.py#L134-L141

et un test aussi lié : https://github.com/zestedesavoir/zds-site/blob/75989e6a27431451effbedb725fb0ff2cbc07f4b/zds/tutorialv2/tests/tests_front.py#L147-L174

Arnaud-D commented 1 month ago

Voir aussi : https://zestedesavoir.com/forums/sujet/1922/une-version-plus-recente-existe/

Arnaud-D commented 1 month ago

Une idée connexe intéressante, avec possibilité de fusionner les deux versions : #3251