entrouvert
/
publik-base-theme
15
1
Fork 1
Code Issues Pull Requests 9 Releases Wiki Activity

toulouse-2022 [parsifal] manage error on templates (#86146) #500

Merged
nroche merged 1 commits from wip/86146-parsifal-manage-err-on-templates into main 2024-02-05 12:02:41 +01:00
Conversation 2 Commits 1 Files Changed 2 +164 -144
nroche commented 2024-01-26 15:24:14 +01:00
Owner
Copy Link

Ajouter la gestion des erreurs sur les 2 templates parsifal présents dans le thème.

Désolé le patch est illisible.
Pourtant, il ne fait qu'ajouter que ces lignes au début des 2 templates :

{% if json.err != 0 %}
  {% if json.err_desc %}
    {{ json.err_desc }}
  {% elif json.data.soap_fault %}
    <pre>{{ json.data }}</pre>
  {% else %}
    Erreur : c'est la tuile (rose)
    <pre>{{ json.data }}</pre>
  {% endif %}
{% else %}
...
{% endif %}
Ajouter la gestion des erreurs sur les 2 templates parsifal présents dans le thème. Désolé le patch est illisible. Pourtant, il ne fait qu'ajouter que ces lignes au début des 2 templates : ``` {% if json.err != 0 %} {% if json.err_desc %} {{ json.err_desc }} {% elif json.data.soap_fault %} <pre>{{ json.data }}</pre> {% else %} Erreur : c'est la tuile (rose) <pre>{{ json.data }}</pre> {% endif %} {% else %} ... {% endif %} ```
nroche added 1 commit 2024-01-26 15:24:14 +01:00
gitea/publik-base-theme/pipeline/head This commit looks good Details
0ccb8274bc toulouse-2022 [parsifal] manage error on templates (#86146)
nroche commented 2024-01-26 15:27:12 +01:00
Author
Owner
Copy Link

Le patch prévoit aussi d'afficher aussi l'erreur SOAP qui a permis de voir le problème ici.

Le patch prévoit aussi d'afficher aussi l'erreur SOAP qui a permis de voir le problème ici.
Screenshot 2024-01-26 at 15-18-13 Panier maélis DEV.png
22 KiB
Screenshot 2024-01-26 at 15-18-13 Panier maélis DEV.png
bdauvergne commented 2024-01-26 20:28:55 +01:00
Owner
Copy Link

Le patch est très lisible si tu cliques sur "ignorer les espaces" dans la vue du diff.

Le patch est très lisible si tu cliques sur "ignorer les espaces" dans la vue du diff.
bdauvergne requested changes 2024-01-26 20:33:11 +01:00
templates/variants/toulouse-2022/combo/json/toulouse-maelis-basket.html Outdated
@ -23,0 +5,4 @@
{% elif json.data.soap_fault %}
<pre>{{ json.data }}</pre>
{% else %}
Erreur : c'est la tuile (rose)
bdauvergne commented 2024-01-26 20:33:05 +01:00
Owner
Copy Link

Il faut prévoir un vrai message pour les usagers ("Une erreur s'est produite veuillez recommencer/recharger la page." et ne pas afficher la soapfault que si {% if is_staff %} je pense on ne sait jamais ce qu'on peut y trouver.

Il faut prévoir un vrai message pour les usagers ("Une erreur s'est produite veuillez recommencer/recharger la page." et ne pas afficher la soapfault que si {% if is_staff %} je pense on ne sait jamais ce qu'on peut y trouver.
bdauvergne commented 2024-01-27 11:23:03 +01:00
Owner
Copy Link

Il faut prévoir un vrai message pour les usagers ("Une erreur s'est produite veuillez recommencer/recharger la page." et ne pas afficher la soapfault que si {% if is_staff %} je pense on ne sait jamais ce qu'on peut y trouver.

Et afficher json et pas json.data dans le <pre></pre>, je vais ouvrir un ticket pour retirer la clé data des exceptions on constate avec ce ticket que c'est perturbant pour les gens qui font les templates.

> Il faut prévoir un vrai message pour les usagers ("Une erreur s'est produite veuillez recommencer/recharger la page." et ne pas afficher la soapfault que si {% if is_staff %} je pense on ne sait jamais ce qu'on peut y trouver. Et afficher json et pas json.data dans le `<pre></pre>`, je vais ouvrir un ticket pour retirer la clé data des exceptions on constate avec ce ticket que c'est perturbant pour les gens qui font les templates.
nroche commented 2024-01-29 10:01:21 +01:00
Author
Owner
Copy Link

Oui, pas pro, déso. Et merci pour is_staff.

{% if json.err != 0 %}
  {% if json.err_desc %}
    {{ json.err_desc }}
  {% elif user.is_staff %}
    <pre>{{ json|pprint }}</pre>
  {% else %}
    Une erreur s'est produite, veuillez recharger la page.
  {% endif %}
{% else %}...
Oui, pas pro, déso. Et merci pour is_staff. ``` {% if json.err != 0 %} {% if json.err_desc %} {{ json.err_desc }} {% elif user.is_staff %} <pre>{{ json|pprint }}</pre> {% else %} Une erreur s'est produite, veuillez recharger la page. {% endif %} {% else %}... ```
nroche force-pushed wip/86146-parsifal-manage-err-on-templates from 0ccb8274bc to 4e6af040f0 2024-01-29 09:59:41 +01:00 Compare
nroche requested review from bdauvergne 2024-01-29 10:01:28 +01:00
bdauvergne approved these changes 2024-02-02 20:46:13 +01:00
nroche force-pushed wip/86146-parsifal-manage-err-on-templates from 4e6af040f0 to bbecc2ec19 2024-02-05 11:58:54 +01:00 Compare
nroche merged commit bbecc2ec19 into main 2024-02-05 12:02:41 +01:00
nroche deleted branch wip/86146-parsifal-manage-err-on-templates 2024-02-05 12:02:41 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
bdauvergne
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/publik-base-theme#500
No description provided.
Delete Branch "wip/86146-parsifal-manage-err-on-templates"

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?