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

misc: add submission context details on front form pages (#9203) #1243

Merged
fpeters merged 1 commits from wip/9203-submission-context-info-front into main 2024-03-15 07:38:09 +01:00
Conversation 0 Commits 1 Files Changed 2 +78 -2
fpeters commented 2024-03-10 13:09:22 +01:00
Owner
Copy Link

Avec ce qui est désormais disponible en variables, on pourrait étendre le message par défaut. À noter que ça peut déjà être ajusté via le paramétrage dans le backoffice et on peut très bien se dire que c'est suffisant et rejeter la PR/le ticket.

Avec ce qui est désormais disponible en variables, on pourrait étendre le message par défaut. À noter que ça peut déjà être ajusté via le paramétrage dans le backoffice et on peut très bien se dire que c'est suffisant et rejeter la PR/le ticket.
fpeters force-pushed wip/9203-submission-context-info-front from 64fa5f3080 to b6e2910e42 2024-03-10 13:38:51 +01:00 Compare
fpeters changed title from WIP: misc: add submission context details on front form pages (#9203) to misc: add submission context details on front form pages (#9203) 2024-03-10 17:44:57 +01:00
tnoel approved these changes 2024-03-11 09:51:20 +01:00
tnoel left a comment
Owner
Copy Link

Plus une question fonctionnelle qu'on relecture technique, est-ce qu'on voudrait ici gérer include_authors_in_form_history et ne pas afficher le nom de l'agent dans ce cas.

Plus une question fonctionnelle qu'on relecture technique, est-ce qu'on voudrait ici gérer include_authors_in_form_history et ne pas afficher le nom de l'agent dans ce cas.
wcs/forms/root.py
@ -2474,0 +2474,4 @@
'''
The form has been recorded on {{ form_receipt_datetime }} with the number {{ form_number }}.
{% if form_submission_agent_display_name %}
It has been submitted for you by {{ form_submission_agent_display_name }}
tnoel commented 2024-03-11 09:50:07 +01:00
Owner
Copy Link

Est-ce qu'on ne voudrait pas ici considérer include_authors_in_form_history ? (ne pas afficher le nom de l'agent)

Est-ce qu'on ne voudrait pas ici considérer include_authors_in_form_history ? (ne pas afficher le nom de l'agent)
fpeters commented 2024-03-15 07:38:03 +01:00
Author
Owner
Copy Link

J'ai poussé sans ça parce qu'on n'a actuellement pas include_authors_in_form_history dans le contexte et que ça alourdissait le gabarit qui doit pourtant pouvoir servir de modèle. (et mettre la variable dans le contexte ça allait faire qu'elle allait être utilisée, alors que je reprendrais bien à un moment tout ça, pour gérer de manière moins bricolée, avec un paramètre par démarche).

J'ai poussé sans ça parce qu'on n'a actuellement pas include_authors_in_form_history dans le contexte et que ça alourdissait le gabarit qui doit pourtant pouvoir servir de modèle. (et mettre la variable dans le contexte ça allait faire qu'elle allait être utilisée, alors que je reprendrais bien à un moment tout ça, pour gérer de manière moins bricolée, avec un paramètre par démarche).
wcs/forms/root.py
@ -2481,0 +2495,4 @@
'''
The form has been recorded on {{ form_receipt_datetime }}.
{% if form_submission_agent_display_name %}
It has been submitted for you by {{ form_submission_agent_display_name }}
tnoel commented 2024-03-11 09:50:24 +01:00
Owner
Copy Link

Même chose, voudrait-on ici considérer include_authors_in_form_history

Même chose, voudrait-on ici considérer include_authors_in_form_history
fpeters merged commit f1471ca20c into main 2024-03-15 07:38:09 +01:00
fpeters deleted branch wip/9203-submission-context-info-front 2024-03-15 07:38:09 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
tnoel
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#1243
No description provided.
Delete Branch "wip/9203-submission-context-info-front"

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?