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

api: add support for temporary access to formdata in backoffice (#22280) #817

Merged
fpeters merged 1 commits from wip/22280-tracking-code-api-backoffice into main 2023-11-10 08:58:56 +01:00
Conversation 0 Commits 1 Files Changed 4 +25 -6
fpeters commented 2023-11-06 16:46:15 +01:00
Owner
Copy Link

Il y aura ensuite à ajuster le "moteur de recherche" code de suivi dans combo pour passer ?backoffice=true.

Aussi, pour un accès backoffice, je zappe la partie "contrôle de champs de la demande", ça se discute, il me semble que l'exploiter pourrait freiner son adoption (parce que "les agents ne peuvent plus aider").

Il y aura ensuite à ajuster le "moteur de recherche" code de suivi dans combo pour passer ?backoffice=true. Aussi, pour un accès backoffice, je zappe la partie "contrôle de champs de la demande", ça se discute, il me semble que l'exploiter pourrait freiner son adoption (parce que "les agents ne peuvent plus aider").
fpeters changed title from WIP: api: add support for temporary access to formdata in backoffice (#22280) to api: add support for temporary access to formdata in backoffice (#22280) 2023-11-06 17:06:43 +01:00
tnoel requested changes 2023-11-07 10:17:24 +01:00
tests/api/test_all.py Outdated
@ -105,0 +109,4 @@
resp = get_url('/api/code/%s?backoffice=true' % code.id, status=200)
assert resp.json['err'] == 0
assert resp.json['url'] == 'http://example.net/backoffice/management/test/%s/' % formdata.id
assert get_app(pub).get(resp.json['load_url']).location == formdata.get_backoffice_url()
tnoel commented 2023-11-07 10:17:13 +01:00
Owner
Copy Link

Je verrais bien à la suite ici un test qui vérifie qu'un accès au formulaire en backoffice affiche bien le texte "This form has been accessed via its tracking code" (qui s'affiche quand un agent accès à un formulaire via le code de suivi). Histoire de vérifier qu'on est passé par le mark_anonymous_formdata

Genre :

...
resp = get_app(pub).get(resp.json['load_url'])
assert resp.location == formdata.get_backoffice_url()
resp.follow()
assert 'This form has been accessed via its tracking code' in resp.text
Je verrais bien à la suite ici un test qui vérifie qu'un accès au formulaire en backoffice affiche bien le texte "This form has been accessed via its tracking code" (qui s'affiche quand un agent accès à un formulaire via le code de suivi). Histoire de vérifier qu'on est passé par le mark_anonymous_formdata Genre : ``` ... resp = get_app(pub).get(resp.json['load_url']) assert resp.location == formdata.get_backoffice_url() resp.follow() assert 'This form has been accessed via its tracking code' in resp.text ```
fpeters commented 2023-11-08 19:34:48 +01:00
Author
Owner
Copy Link

Ça a demandé un peu plus que ces 4 lignes mais ça a été ajouté au test.

Ça a demandé un peu plus que ces 4 lignes mais ça a été ajouté au test.
tnoel marked this conversation as resolved
fpeters force-pushed wip/22280-tracking-code-api-backoffice from 1478b234bb to bdfd1e93ec 2023-11-07 14:11:06 +01:00 Compare
fpeters requested review from tnoel 2023-11-08 19:34:54 +01:00
tnoel approved these changes 2023-11-09 10:47:04 +01:00
fpeters merged commit de0c4cb26a into main 2023-11-10 08:58:56 +01:00
fpeters deleted branch wip/22280-tracking-code-api-backoffice 2023-11-10 08:58:56 +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#817
No description provided.
Delete Branch "wip/22280-tracking-code-api-backoffice"

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?