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

templatetags: add {% make_public_url %} (#83226) #819

Merged
fpeters merged 1 commits from wip/83226-make-public-url into main 2023-11-10 08:57:27 +01:00
Conversation 1 Commits 1 Files Changed 4 +84 -3
fpeters commented 2023-11-08 15:50:11 +01:00
Owner
Copy Link

Le ticket parle d'un cas spécifique où en front l'usager concerné aurait le bon rôle pour voir l'image associée mais en pratique souvent ça ne sera pas le cas; pour répondre également à la situation où il n'y aurait pas d'accès, et pour ainsi faire comme côté combo, j'ajoute {% make_public_url %}. (côté combo il y a un rebond par une url locale qui fait la signature, ici je pose directement la signature).

Le ticket parle d'un cas spécifique où en front l'usager concerné aurait le bon rôle pour voir l'image associée mais en pratique souvent ça ne sera pas le cas; pour répondre également à la situation où il n'y aurait pas d'accès, et pour ainsi faire comme côté combo, j'ajoute `{% make_public_url %}`. (côté combo il y a un rebond par une url locale qui fait la signature, ici je pose directement la signature).
fpeters force-pushed wip/83226-make-public-url from aecc6543d7 to 54cfcac515 2023-11-08 18:13:19 +01:00 Compare
fpeters reviewed 2023-11-08 19:27:44 +01:00
tests/form_pages/test_live.py Outdated
@ -2365,6 +2366,61 @@ def test_comment_from_card_field(pub):
assert live_resp.json['result']['3']['content'] == '<p>Xbar {{ form_var_foo }}Ybar plopZ</p>'
def test_comment_from_card_field_image_url(pub):
fpeters commented 2023-11-08 19:27:44 +01:00
Author
Owner
Copy Link

Un test qui correspond précisément à l'utilisation régulièrement souhaitée, avoir une image issue d'une fiche affichée lors de la saisie d'une demande.

Un test qui correspond précisément à l'utilisation régulièrement souhaitée, avoir une image issue d'une fiche affichée lors de la saisie d'une demande.
fpeters changed title from WIP: templatetags: add {% make_public_url %} (#83226) to templatetags: add {% make_public_url %} (#83226) 2023-11-08 19:27:54 +01:00
lguerin approved these changes 2023-11-09 09:31:59 +01:00
tnoel reviewed 2023-11-09 09:59:40 +01:00
wcs/qommon/templatetags/qommon.py Outdated
@ -1238,0 +1240,4 @@
@register.simple_tag(takes_context=True)
def make_public_url(context, url):
return sign_url_auto_orig(url or '')
tnoel commented 2023-11-09 09:59:25 +01:00
Owner
Copy Link

Je ne sais pas dans quelle mesure on pourrait dire que cette URL aura une durée de vie de 30 secondes... on peut sans doute passer cela sous silence pour l'instant, et si ça pose pépin un jour (genre, dans un mail), on corrigera...

Je ne sais pas dans quelle mesure on pourrait dire que cette URL aura une durée de vie de 30 secondes... on peut sans doute passer cela sous silence pour l'instant, et si ça pose pépin un jour (genre, dans un mail), on corrigera...
fpeters commented 2023-11-09 10:11:08 +01:00
Author
Owner
Copy Link

Je ne sais pas dans quelle mesure on pourrait dire que cette URL aura une durée de vie de 30 secondes... on peut sans doute passer cela sous silence pour l'instant, et si ça pose pépin un jour (genre, dans un mail), on corrigera...

Dans le ticket Lauréline a pointé #78711 et cette durée courte ne le résoudrait pas, j'imaginais d'abord faire un nouveau ticket sur le sujet mais là je viens de pousser un commit supplémentaire, pour faire sans la durée courte (avec une url liée à la session en cours). (build pas encore terminé, si les tests échouent je corrigerai etc.)

> Je ne sais pas dans quelle mesure on pourrait dire que cette URL aura une durée de vie de 30 secondes... on peut sans doute passer cela sous silence pour l'instant, et si ça pose pépin un jour (genre, dans un mail), on corrigera... Dans le ticket Lauréline a pointé #78711 et cette durée courte ne le résoudrait pas, j'imaginais d'abord faire un nouveau ticket sur le sujet mais là je viens de pousser un commit supplémentaire, pour faire sans la durée courte (avec une url liée à la session en cours). (build pas encore terminé, si les tests échouent je corrigerai etc.)
fpeters force-pushed wip/83226-make-public-url from db01fe53b7 to 8b4d40762f 2023-11-09 10:24:25 +01:00 Compare
fpeters commented 2023-11-09 10:39:48 +01:00
Author
Owner
Copy Link

Voilà, 8b4d40762f plutôt qu'un nouveau ticket ça m'irait de directement passer par la relecture ici.

Voilà, https://git.entrouvert.org/entrouvert/wcs/commit/8b4d40762fd1a394df1b1a918d2fc3a97bb7fd18 plutôt qu'un nouveau ticket ça m'irait de directement passer par la relecture ici.
fpeters requested review from lguerin 2023-11-09 10:39:53 +01:00
lguerin approved these changes 2023-11-09 11:04:33 +01:00
fpeters force-pushed wip/83226-make-public-url from 8b4d40762f to 622fa3d12e 2023-11-09 15:34:07 +01:00 Compare
fpeters merged commit f02596a7e8 into main 2023-11-10 08:57:27 +01:00
fpeters deleted branch wip/83226-make-public-url 2023-11-10 08:57:27 +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
3 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#819
No description provided.
Delete Branch "wip/83226-make-public-url"

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?