misc: add proper message in case of double submission (#66919) #975

Merged
fpeters merged 3 commits from wip/66919-submitted-draft-exception-on-previous-page into main 2024-01-12 14:53:39 +01:00
Owner
No description provided.
fpeters force-pushed wip/66919-submitted-draft-exception-on-previous-page from 412c79756d to c0ad790738 2023-12-29 14:18:13 +01:00 Compare
fpeters changed title from WIP: misc: add proper message in case of double submission (#66919) to misc: add proper message in case of double submission (#66919) 2024-01-05 18:54:40 +01:00
pmarillonnet approved these changes 2024-01-08 09:02:18 +01:00
pmarillonnet left a comment
Owner

Deux petits trucs ramassés au passage, à prendre ou pas. Ack.

Deux petits trucs ramassés au passage, à prendre ou pas. Ack.
@ -1145,0 +1143,4 @@
# first step,
if not (get_session().user or get_session().anonymous_formdata_keys):
# that means we probably lost the session in mid-air.
get_session().message = ('error', _('Sorry, your session have been lost.'))
Owner

Petite coquille ici (your session has […]).

Petite coquille ici (your session *has* […]).
Author
Owner

Merci j'ai ajouté un commit pour corriger cette erreur qui est là depuis 2015…

Merci j'ai ajouté un commit pour corriger cette erreur qui est là depuis 2015…
@ -1381,7 +1386,13 @@ class FormPage(Directory, TempfileDirectoryMixin, FormTemplateMixin):
if form.has_errors() or form.get_submit() is True:
if self.has_draft_support() and not honeypot_error:
# always save draft during server roundtrip, even if the form has errors
Owner

Du détail, mais question de lisibilité je laisserais ici un commentaire du genre de ce qui est plus bas pour l’autosave, # save draft during server roundtrip, je trouve le always … even if … moins justifié maintenant qu’on prend des précautions supplémentaires à la tentative de sauvegarde.

Du détail, mais question de lisibilité je laisserais ici un commentaire du genre de ce qui est plus bas pour l’autosave, ` # save draft during server roundtrip`, je trouve le `always … even if …` moins justifié maintenant qu’on prend des précautions supplémentaires à la tentative de sauvegarde.
Author
Owner

Ok j'ai modifié le commentaire, sans arriver à reformuler l'idée que ça n'était pas grave que des données en erreur soient enregistrées.

Ok j'ai modifié le commentaire, sans arriver à reformuler l'idée que ça n'était pas grave que des données en erreur soient enregistrées.
fpeters force-pushed wip/66919-submitted-draft-exception-on-previous-page from c0ad790738 to 0bec3c80f2 2024-01-08 09:55:46 +01:00 Compare
fpeters force-pushed wip/66919-submitted-draft-exception-on-previous-page from 0bec3c80f2 to 7bcef49a45 2024-01-08 10:05:49 +01:00 Compare
fpeters merged commit 14fda2596a into main 2024-01-12 14:53:39 +01:00
fpeters deleted branch wip/66919-submitted-draft-exception-on-previous-page 2024-01-12 14:53:39 +01:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
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#975
No description provided.