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

split pending submissions and use a table (#13415) #1187

Merged
fpeters merged 2 commits from wip/13415-backoffice-submission-table into main 2024-02-29 19:10:20 +01:00
Conversation 1 Commits 2 Files Changed 3 +212 -161
fpeters commented 2024-02-24 17:32:33 +01:00
Owner
Copy Link

Les commentaires de perf sur l'écran de saisie ramènent à ce ticket ("faire l'écran de saisie backoffice sous forme de listing"), ici,

  • 1/ séparer l'écran "nouvelle saisie" de l'écran "saisie en cours"
  • 2/ utiliser un tableau pour l'écran "saisie en cours"

Ça élimine en passant la notion de "prédemande" qui était quelque chose de welco (un brouillon de demande créé par welco, et encodé ensuite par un autre agent dans wcs).

Les commentaires de perf sur l'écran de saisie ramènent à ce ticket ("faire l'écran de saisie backoffice sous forme de listing"), ici, * 1/ séparer l'écran "nouvelle saisie" de l'écran "saisie en cours" * 2/ utiliser un tableau pour l'écran "saisie en cours" Ça élimine en passant la notion de "prédemande" qui était quelque chose de welco (un brouillon de demande créé par welco, et encodé ensuite par un autre agent dans wcs).
fpeters added 2 commits 2024-02-24 17:32:33 +01:00
fpeters force-pushed wip/13415-backoffice-submission-table from 577186614f to c892c231ee 2024-02-24 17:51:41 +01:00 Compare
fpeters changed target branch from main to wip/87435-submission-index-query 2024-02-24 17:53:10 +01:00
fpeters force-pushed wip/13415-backoffice-submission-table from c892c231ee to 719e4bf477 2024-02-24 18:08:45 +01:00 Compare
bdauvergne requested changes 2024-02-27 18:46:26 +01:00
Dismissed
bdauvergne left a comment
Owner
Copy Link

On perd au passage la distinction has_empty_date() oui/non mais je suppose que c'est voulu (je ne voyais pas bien l'usage non plus) et donc on peut utiliser AnyFornData c'est ok.

On perd au passage la distinction has_empty_date() oui/non mais je suppose que c'est voulu (je ne voyais pas bien l'usage non plus) et donc on peut utiliser AnyFornData c'est ok.
wcs/backoffice/submission.py Outdated
@ -570,0 +582,4 @@
rt = TemplateIO(html=True)
rt += htmltext('<div id="appbar">')
rt += htmltext('<h2>%s</h2>') % _('Pending submissions')
rt += htmltext('</span>')
bdauvergne commented 2024-02-27 18:42:40 +01:00
Owner
Copy Link

Ce span fermant m'a l'air tout seul.

Ce span fermant m'a l'air tout seul.
fpeters commented 2024-02-28 10:33:16 +01:00
Author
Owner
Copy Link

On perd au passage la distinction has_empty_date() oui/non mais je suppose que c'est voulu (je ne voyais pas bien l'usage non plus) et donc on peut utiliser AnyFornData c'est ok.

Oui c'est la notion de "prédemande" dont je parlais.

> On perd au passage la distinction has_empty_date() oui/non mais je suppose que c'est voulu (je ne voyais pas bien l'usage non plus) et donc on peut utiliser AnyFornData c'est ok. Oui c'est la notion de "prédemande" dont je parlais.
fpeters closed this pull request 2024-02-28 10:35:56 +01:00
fpeters reopened this pull request 2024-02-28 13:06:04 +01:00
fpeters changed target branch from wip/87435-submission-index-query to main 2024-02-28 13:06:11 +01:00
fpeters force-pushed wip/13415-backoffice-submission-table from f35c955c1d to 87e2314d7b 2024-02-28 13:07:51 +01:00 Compare
fpeters force-pushed wip/13415-backoffice-submission-table from 87e2314d7b to 6001ff8c4e 2024-02-28 13:10:02 +01:00 Compare
fpeters dismissed bdauvergne’s review 2024-02-29 17:03:27 +01:00
Reason:

(prise en compte)

fpeters changed title from WIP: split pending submissions and use a table (#13415) to split pending submissions and use a table (#13415) 2024-02-29 17:03:38 +01:00
bdauvergne approved these changes 2024-02-29 17:26:25 +01:00
fpeters merged commit f4e9e7d3ac into main 2024-02-29 19:10:20 +01:00
fpeters deleted branch wip/13415-backoffice-submission-table 2024-02-29 19:10:20 +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/wcs#1187
No description provided.
Delete Branch "wip/13415-backoffice-submission-table"

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?