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

api: add support for include-anonymised=on/off in form/card lists (#86603) #1097

Merged
fpeters merged 2 commits from wip/86603-include-anonymised into main 2024-02-09 07:27:48 +01:00
Conversation 0 Commits 2 Files Changed 4 +16 -2
fpeters commented 2024-02-06 13:21:58 +01:00
Owner
Copy Link
No description provided.
fpeters force-pushed wip/86603-include-anonymised from 0bcf24389b to 6beb276135 2024-02-06 13:51:31 +01:00 Compare
fpeters reviewed 2024-02-06 14:33:20 +01:00
wcs/api.py
@ -507,3 +507,3 @@
if not get_query_flag('include-anonymised', default=True):
if not get_query_flag('include-anonymised', default=False):
criterias.append(Null('anonymised'))
fpeters commented 2024-02-06 14:22:50 +01:00
Author
Owner
Copy Link

Dans un deuxième commit je modifie /api/forms pour avoir le même comportement de ne pas prendre les demandes anonymisées par défaut; il y a un appel à cette API dans combo, elle passe déjà explicitement ?include-anonymised=off.

Dans un deuxième commit je modifie /api/forms pour avoir le même comportement de ne pas prendre les demandes anonymisées par défaut; il y a un appel à cette API dans combo, elle passe déjà explicitement ?include-anonymised=off.
wcs/backoffice/management.py
@ -2707,2 +2707,4 @@
limit = misc.get_int_or_400(get_request().form['limit'])
if not get_query_flag('include-anonymised', default=False):
criterias.append(Null('anonymised'))
fpeters commented 2024-02-06 14:32:46 +01:00
Author
Owner
Copy Link

Ici la demande initiale, pour ne pas reprendre par défaut les demandes/fiches anonymisées.

Évidemment ça change le comportement actuel qui était de retourner ce qui était anonymisé, et ça a un impact côté wcs-olap (j'ai créé #86606).

Ici la demande initiale, pour ne pas reprendre par défaut les demandes/fiches anonymisées. Évidemment ça change le comportement actuel qui était de retourner ce qui était anonymisé, et ça a un impact côté wcs-olap (j'ai créé #86606).
fpeters changed title from WIP: api: add support for include-anonymised=on/off in form/card lists (#86603) to api: add support for include-anonymised=on/off in form/card lists (#86603) 2024-02-06 14:36:26 +01:00
tnoel approved these changes 2024-02-06 15:46:31 +01:00
fpeters merged commit 108b249965 into main 2024-02-09 07:27:48 +01:00
fpeters deleted branch wip/86603-include-anonymised 2024-02-09 07:27:48 +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#1097
No description provided.
Delete Branch "wip/86603-include-anonymised"

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?