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

forms: add exception string to technical error message (#63776) #815

Merged
fpeters merged 1 commits from wip/63776-technical-error-details into main 2023-11-10 08:58:10 +01:00
Conversation 1 Commits 1 Files Changed 13 +73 -39
fpeters commented 2023-11-05 11:25:07 +01:00
Owner
Copy Link

image

qui se déplie,

image

![image](/attachments/d9d68df1-db21-4cc8-beec-f7a49885fbdd) qui se déplie, ![image](/attachments/e7ae88fc-ae2b-40b8-ab80-98a70a85678b)
image.png
5.8 KiB
image.png
11 KiB
fpeters force-pushed wip/63776-technical-error-details from 85db60f58a to 4d9214b2c9 2023-11-05 12:05:15 +01:00 Compare
fpeters force-pushed wip/63776-technical-error-details from 4d9214b2c9 to 62c186355e 2023-11-05 13:37:39 +01:00 Compare
fpeters reviewed 2023-11-05 14:01:20 +01:00
wcs/fields/base.py Outdated
@ -627,3 +627,3 @@
display_value = self.store_display_value(data, self.id)
if raise_on_error and display_value is None:
raise SetValueError('a datasource is unavailable (field id: %s)' % self.id)
raise SetValueError(_('datasource is unavailable (field id: %s)') % self.id)
fpeters commented 2023-11-05 13:58:24 +01:00
Author
Owner
Copy Link

Je marque les chaines pour traduction vu qu'elles apparaitront dans l'interface.

Je marque les chaines pour traduction vu qu'elles apparaitront dans l'interface.
wcs/forms/common.py Outdated
@ -331,3 +331,2 @@
except RedisplayFormException:
# don't display errors after "add block" button has been clicked.
form.clear_errors()
pass
fpeters commented 2023-11-05 13:58:58 +01:00
Author
Owner
Copy Link

Cette partie se trouve intégrée dans l'exception, pour éviter la duplication.

Cette partie se trouve intégrée dans l'exception, pour éviter la duplication.
wcs/qommon/form.py Outdated
@ -500,0 +501,4 @@
htmltext(
'<details><summary>%(summary)s</summary><p><small>%(details)s</small></p></details>'
)
% error
fpeters commented 2023-11-05 14:00:13 +01:00
Author
Owner
Copy Link

On peut passer en erreur un dictionnaire, {"summary": ..., "details": ...} et ça sera affiché dans une zone dépliable.

On peut passer en erreur un dictionnaire, {"summary": ..., "details": ...} et ça sera affiché dans une zone dépliable.
fpeters force-pushed wip/63776-technical-error-details from 62c186355e to 308f7b38da 2023-11-05 14:01:25 +01:00 Compare
fpeters reviewed 2023-11-05 14:02:08 +01:00
wcs/wf/form.py
@ -331,1 +329,3 @@
raise RedisplayFormException()
except SetValueError as e:
raise RedisplayFormException(
form=form, error={'summary': _('Technical error, please try again.'), 'details': e}
fpeters commented 2023-11-05 14:02:08 +01:00
Author
Owner
Copy Link

Plutôt qu'utiliser l'infrastructure générale d'affichage de message de session, ça va aller le mettre correctement en erreur sur le formulaire.

Plutôt qu'utiliser l'infrastructure générale d'affichage de message de session, ça va aller le mettre correctement en erreur sur le formulaire.
fpeters commented 2023-11-05 14:04:10 +01:00
Author
Owner
Copy Link

Avec ça j'ai aussi un mini-patch côté publik-base-theme, pour faire :

--- a/static/includes/_misc.scss
+++ b/static/includes/_misc.scss
@@ -396,6 +396,10 @@ div#page-change-overlay.on {
                        margin-bottom: 0;
                }
        }
+
+       details {
+               padding: 0.3em 0;
+       }
 }

ça permet au texte "technical error, pleasy try again" de ne pas changer de position quand c'est déplié.

Avec ça j'ai aussi un mini-patch côté publik-base-theme, pour faire : ``` --- a/static/includes/_misc.scss +++ b/static/includes/_misc.scss @@ -396,6 +396,10 @@ div#page-change-overlay.on { margin-bottom: 0; } } + + details { + padding: 0.3em 0; + } } ``` ça permet au texte "technical error, pleasy try again" de ne pas changer de position quand c'est déplié.
fpeters changed title from WIP: forms: add exception string to technical error message (#63776) to forms: add exception string to technical error message (#63776) 2023-11-05 14:20:05 +01:00
lguerin approved these changes 2023-11-06 10:26:28 +01:00
fpeters merged commit 7df88dbb6d into main 2023-11-10 08:58:10 +01:00
fpeters deleted branch wip/63776-technical-error-details 2023-11-10 08:58:10 +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#815
No description provided.
Delete Branch "wip/63776-technical-error-details"

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?