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

misc: serialize draft after select2 fields have been initialized (#39399) #1248

Merged
fpeters merged 1 commits from wip/39399-draft-autosave-no-change into main 2024-03-15 07:20:58 +01:00
Conversation 0 Commits 1 Files Changed 1 +3 -1
fpeters commented 2024-03-11 15:01:07 +01:00
Owner
Copy Link

Rencontré en travaillant sur un autre ticket; ce qu'il se passe c'est que la sérialisation initiale attrape des champs qui sont ensuite/immédiatement transformés par select2, ce qui fait que même sur les données initiales il y a un autosave.

Le ticket à l'origine pointe un formulaire qui n'existe plus mais pour l'autre ticket lié il y a biens des select2 sur la page, donc ça se tient.

Rencontré en travaillant sur un autre ticket; ce qu'il se passe c'est que la sérialisation initiale attrape des champs qui sont ensuite/immédiatement transformés par select2, ce qui fait que même sur les données initiales il y a un autosave. Le ticket à l'origine pointe un formulaire qui n'existe plus mais pour l'autre ticket lié il y a biens des select2 sur la page, donc ça se tient.
fpeters added 1 commit 2024-03-11 15:01:07 +01:00
fpeters changed title from WIP: misc: serialize draft after select2 fields have been initialized (#39399) to misc: serialize draft after select2 fields have been initialized (#39399) 2024-03-11 15:01:10 +01:00
fpeters reviewed 2024-03-11 15:06:14 +01:00
wcs/qommon/static/js/qommon.forms.js
@ -150,3 +150,3 @@
var last_auto_save = $('form[data-has-draft]').serialize();
if ($('form[data-has-draft]:not([data-autosave=false])').length == 1) {
var last_auto_save = $('form[data-has-draft]').serialize();
fpeters commented 2024-03-11 15:05:39 +01:00
Author
Owner
Copy Link

Je ne suis pas totalement sûr que ça soit nécessaire d'initialiser avec une valeur dès le début mais ça ne coûte pas grand chose.

Je ne suis pas totalement sûr que ça soit nécessaire d'initialiser avec une valeur dès le début mais ça ne coûte pas grand chose.
wcs/qommon/static/js/qommon.forms.js
@ -439,6 +440,7 @@ $(function() {
}
add_js_behaviours($('form[data-live-url], form[data-backoffice-preview]'));
last_auto_save = $('form[data-has-draft]').serialize();
fpeters commented 2024-03-11 15:06:10 +01:00
Author
Owner
Copy Link

Et un peu plus tard, après les select2 et l'initialisation des js de manière plus générale, actualisation du contenu.

Et un peu plus tard, après les select2 et l'initialisation des js de manière plus générale, actualisation du contenu.
lguerin approved these changes 2024-03-12 08:58:23 +01:00
fpeters merged commit a1eb55d19e into main 2024-03-15 07:20:58 +01:00
fpeters deleted branch wip/39399-draft-autosave-no-change 2024-03-15 07:20:58 +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#1248
No description provided.
Delete Branch "wip/39399-draft-autosave-no-change"

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?