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

"internal_id" dans le json des fiches (#87117) #1163

Merged
fpeters merged 2 commits from wip/87117-json-card into main 2024-02-23 19:05:07 +01:00
Conversation 0 Commits 2 Files Changed 5 +19 -9
fpeters commented 2024-02-19 10:17:24 +01:00
Owner
Copy Link
No description provided.
fpeters force-pushed wip/87117-json-card from 25c93a49c3 to f5d364c167 2024-02-19 10:33:04 +01:00 Compare
fpeters force-pushed wip/87117-json-card from f5d364c167 to 1b8cb2cb8c 2024-02-19 10:39:15 +01:00 Compare
fpeters reviewed 2024-02-19 12:28:41 +01:00
tests/api/test_carddata.py
@ -467,3 +467,3 @@
carddef = CardDef()
carddef.name = 'foo'
carddef.workflow_roles = {'_receiver': role.id}
carddef.workflow_roles = {'_viewer': role.id}
fpeters commented 2024-02-19 12:26:58 +01:00
Author
Owner
Copy Link

Un premier commit "tests: fix workflow function used in carddata tests (#87117)" parce que le test actuel faisait

    assert len(resp.json) == 1

et ça marchait parce que la réponse était {"data": []}, alors qu'on voulait vérifier le nombre d'éléments dans "data".

Un premier commit "tests: fix workflow function used in carddata tests (#87117)" parce que le test actuel faisait ``` assert len(resp.json) == 1 ``` et ça marchait parce que la réponse était `{"data": []}`, alors qu'on voulait vérifier le nombre d'éléments dans "data".
wcs/backoffice/management.py
@ -2779,2 +2779,3 @@
{
'id': filled.id,
'id': filled.identifier,
'internal_id': str(filled.id),
fpeters commented 2024-02-19 12:28:38 +01:00
Author
Owner
Copy Link

Ce qui était la description initiale du ticket.

Ce qui était la description initiale du ticket.
wcs/formdata.py
@ -1536,3 +1536,3 @@
data['uuid'] = self.uuid
data['id'] = self.identifier
data['internal-id'] = str(self.id)
data['internal_id'] = str(self.id)
fpeters commented 2024-02-19 12:27:47 +01:00
Author
Owner
Copy Link

Ça n'était pas encore exploité, à ma connaissance et en faisant quelques grep, donc pour faciliter l'emploi dans du gabarit django, je passe à l'underscore.

Ça n'était pas encore exploité, à ma connaissance et en faisant quelques grep, donc pour faciliter l'emploi dans du gabarit django, je passe à l'underscore.
fpeters changed title from WIP: "internal_id" dans le json des fiches (#87117) to "internal_id" dans le json des fiches (#87117) 2024-02-19 12:28:48 +01:00
lguerin approved these changes 2024-02-20 11:15:05 +01:00
fpeters merged commit 8ee2a8cee6 into main 2024-02-23 19:05:07 +01:00
fpeters deleted branch wip/87117-json-card 2024-02-23 19:05:07 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
lguerin
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#1163
No description provided.
Delete Branch "wip/87117-json-card"

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?