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

statistics: remove all forms filter option (#80462) #619

Merged
vdeniaud merged 1 commits from wip/80462-stats-enlever-le-choix-Tous-les- into main 2024-04-30 10:45:47 +02:00
Conversation 0 Commits 1 Files Changed 2 +2 -12
vdeniaud commented 2023-08-22 10:31:52 +02:00
Owner
Copy Link

Je laisse en wip en attendant combo.

Je laisse en wip en attendant combo.
vdeniaud added 1 commit 2023-08-22 10:31:53 +02:00
gitea/wcs/pipeline/head This commit looks good Details
331ba0ee0f statistics: remove all forms filter option (#80462)
vdeniaud force-pushed wip/80462-stats-enlever-le-choix-Tous-les- from 331ba0ee0f to 00f84bfa12 2024-04-18 17:23:48 +02:00 Compare
vdeniaud changed title from WIP: statistics: remove all forms filter option (#80462) to statistics: remove all forms filter option (#80462) 2024-04-18 17:24:20 +02:00
ecazenave requested changes 2024-04-18 17:42:10 +02:00
Dismissed
wcs/statistics/views.py Outdated
@ -96,3 +96,2 @@
'label': _('Form'),
'label': _('Form(s)'),
'options': self.get_form_options(FormDef),
'required': True,
ecazenave commented 2024-04-18 17:41:58 +02:00
Owner
Copy Link

Normale que le required saute ?

Normale que le required saute ?
vdeniaud commented 2024-04-18 17:47:47 +02:00
Author
Owner
Copy Link

Oui c'était déjà ceinture et bretelles avant, plus bas on fait

        slugs = request.GET.getlist('form', ['_all'] if self.has_global_count_support else ['_nothing'])

donc pas de pb si ce paramètre est pas là.

Et fonctionnellement c'est parce qu'on remplace le select multiple par un champ select2, et donc au lieu d'avoir « Tous les formulaires » pour filtrer sur rien, on a le choix vide

Oui c'était déjà ceinture et bretelles avant, plus bas on fait ``` slugs = request.GET.getlist('form', ['_all'] if self.has_global_count_support else ['_nothing']) ``` donc pas de pb si ce paramètre est pas là. Et fonctionnellement c'est parce qu'on remplace le select multiple par un champ select2, et donc au lieu d'avoir « Tous les formulaires » pour filtrer sur rien, on a le choix vide
ecazenave approved these changes 2024-04-18 17:55:35 +02:00
vdeniaud force-pushed wip/80462-stats-enlever-le-choix-Tous-les- from 00f84bfa12 to b90e68fc84 2024-04-29 09:46:40 +02:00 Compare
vdeniaud merged commit 23d3483386 into main 2024-04-30 10:45:47 +02:00
vdeniaud deleted branch wip/80462-stats-enlever-le-choix-Tous-les- 2024-04-30 10:45:47 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
ecazenave
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#619
No description provided.
Delete Branch "wip/80462-stats-enlever-le-choix-Tous-les-"

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?