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

backoffice: use sql criterias to get drafts waiting for submission (#87435) #1185

Merged
fpeters merged 2 commits from wip/87435-submission-index-query into main 2024-02-28 10:35:56 +01:00
Conversation 4 Commits 2 Files Changed 1 +23 -21
fpeters commented 2024-02-24 10:54:00 +01:00
Owner
Copy Link
No description provided.
fpeters changed title from WIP: backoffice: use sql criterias to get drafts waiting for submission (#87435) to backoffice: use sql criterias to get drafts waiting for submission (#87435) 2024-02-24 11:09:57 +01:00
fpeters commented 2024-02-24 11:21:17 +01:00
Author
Owner
Copy Link

Ajustements supplémentaires possibles :

  • utiliser sql.AnyFormData
  • prefetch des agents (cf branche de Benjamin).
Ajustements supplémentaires possibles : * utiliser sql.AnyFormData * prefetch des agents (cf branche de Benjamin).
fpeters changed title from backoffice: use sql criterias to get drafts waiting for submission (#87435) to WIP: backoffice: use sql criterias to get drafts waiting for submission (#87435) 2024-02-24 11:21:25 +01:00
bdauvergne commented 2024-02-24 11:40:46 +01:00
Owner
Copy Link

Ajustements supplémentaires possibles :

  • utiliser sql.AnyFormData

Il n'y a pas les données, à moins de retirer le besoin d'appeler FormData.has_empty_data() ça ne sera pas à iso comportement.

> Ajustements supplémentaires possibles : > > * utiliser sql.AnyFormData Il n'y a pas les données, à moins de retirer le besoin d'appeler FormData.has_empty_data() ça ne sera pas à iso comportement.
bdauvergne commented 2024-02-24 11:45:19 +01:00
Owner
Copy Link
  • prefetch des agents (cf branche de Benjamin).

Pour le besoin de prefetch je pense qu'il existe, sur le ticket Toulouse je vois qu'un des formulaires coûteux (requête SQL de 7s) on avait 34000 brouillons simple et là dedans environ 350 brouillons backoffice, donc 350 SELECT sur la table users potentiels.

wcs_demarches_eservices_toulouse_metropole_fr=# select count(*) from formdata_18_modifier_reservation where status = 'draft' and backoffice_submission and submission_agent_id is not null;
 count 
-------
  487
(1 ligne)

wcs_demarches_eservices_toulouse_metropole_fr=# select count(*) from formdata_18_modifier_reservation where status = 'draft';
 count 
-------
 33588
(1 ligne)

Après il est évident que la situation au niveau des brouillons backoffice me semble loin d'être normale.

> * prefetch des agents (cf branche de Benjamin). Pour le besoin de prefetch je pense qu'il existe, sur le ticket Toulouse je vois qu'un des formulaires coûteux (requête SQL de 7s) on avait 34000 brouillons simple et là dedans environ 350 brouillons backoffice, donc 350 SELECT sur la table users potentiels. ``` wcs_demarches_eservices_toulouse_metropole_fr=# select count(*) from formdata_18_modifier_reservation where status = 'draft' and backoffice_submission and submission_agent_id is not null; count ------- 487 (1 ligne) wcs_demarches_eservices_toulouse_metropole_fr=# select count(*) from formdata_18_modifier_reservation where status = 'draft'; count ------- 33588 (1 ligne) ``` Après il est évident que la situation au niveau des brouillons backoffice me semble loin d'être normale.
fpeters force-pushed wip/87435-submission-index-query from 8c0fc4c3c7 to df53ce5df3 2024-02-24 17:32:52 +01:00 Compare
fpeters commented 2024-02-26 12:04:04 +01:00
Author
Owner
Copy Link

Voilà pour le minimum ici; le commentaire sur le nombre de brouillons (même s'il est anormal) m'a fait relancer #13415 pour qu'ils soient présentés dans un tableau, avec pagination etc.

Voilà pour le minimum ici; le commentaire sur le nombre de brouillons (même s'il est anormal) m'a fait relancer #13415 pour qu'ils soient présentés dans un tableau, avec pagination etc.
fpeters changed title from WIP: backoffice: use sql criterias to get drafts waiting for submission (#87435) to backoffice: use sql criterias to get drafts waiting for submission (#87435) 2024-02-26 12:04:07 +01:00
tnoel requested review from bdauvergne 2024-02-27 18:18:42 +01:00
bdauvergne approved these changes 2024-02-27 18:41:14 +01:00
fpeters merged commit 2214a45cde into main 2024-02-28 10:35:56 +01:00
fpeters deleted branch wip/87435-submission-index-query 2024-02-28 10:35:56 +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#1185
No description provided.
Delete Branch "wip/87435-submission-index-query"

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?