entrouvert
/
chrono
14
1
Fork 0
Code Issues Pull Requests 8 Releases Wiki Activity

applification: implémenter le endpoint bundle-check (#87653) #218

Merged
lguerin merged 4 commits from wip/87653-app-bundle-check into main 2024-03-21 13:36:29 +01:00
Conversation 1 Commits 4 Files Changed 4 +624 -3
lguerin commented 2024-03-01 15:13:24 +01:00
Owner
Copy Link
No description provided.
lguerin added 4 commits 2024-03-01 15:13:24 +01:00
lguerin changed title from WIP: applification: implémenter le endpoint bundle-check (#87653) to applification: implémenter le endpoint bundle-check (#87653) 2024-03-01 15:17:28 +01:00
lguerin force-pushed wip/87653-app-bundle-check from a16abbfc75 to d465105fbc 2024-03-07 16:44:49 +01:00 Compare
lguerin changed target branch from wip/87316-snapshot-history-and-compare to main 2024-03-15 08:29:09 +01:00
lguerin force-pushed wip/87653-app-bundle-check from d465105fbc to 51b4892f07 2024-03-15 08:35:13 +01:00 Compare
lguerin force-pushed wip/87653-app-bundle-check from 51b4892f07 to e29172ec74 2024-03-21 10:16:22 +01:00 Compare
pmarillonnet commented 2024-03-21 10:24:50 +01:00
Owner
Copy Link

J’avais relu l’équivalent pour lingo, je vais prendre cette relecture aussi.

J’avais relu l’équivalent pour lingo, je vais prendre cette relecture aussi.
pmarillonnet requested review from pmarillonnet 2024-03-21 10:25:11 +01:00
pmarillonnet reviewed 2024-03-21 11:34:32 +01:00
chrono/apps/export_import/api_views.py Outdated
@ -193,6 +223,7 @@ def component_redirect(request, component_type, slug):
return redirect(reverse('chrono-manager-resource-view', kwargs={'pk': component.pk}))
if klass == UnavailabilityCalendar:
return redirect(reverse('chrono-manager-unavailability-calendar-view', kwargs={'pk': component.pk}))
pmarillonnet commented 2024-03-21 11:16:11 +01:00
Owner
Copy Link

(Je pense un peu de code churn qui a dû avoir lieu ici à un moment de l’écriture de la PR, avec un saut à la ligne qui est finalement resté tout seul. Je serais pour retirer parce que ça opacifie la lecture des git-blame.)

(Je pense un peu de code churn qui a dû avoir lieu ici à un moment de l’écriture de la PR, avec un saut à la ligne qui est finalement resté tout seul. Je serais pour retirer parce que ça opacifie la lecture des git-blame.)
lguerin commented 2024-03-21 11:50:31 +01:00
Author
Owner
Copy Link

corrigé

corrigé
pmarillonnet marked this conversation as resolved
chrono/apps/export_import/api_views.py Outdated
@ -204,0 +276,4 @@
# information needed here, Relation objects may not exist yet in hobo
'text': component.label,
'url': reverse(
'api-export-import-component-redirect',
pmarillonnet commented 2024-03-21 10:53:56 +01:00
Owner
Copy Link

Niveau parcours applification, l’idée de ce component redirect c’est qu’on invite l’utilisateur à lier l’objet à une application existante ? Ou bien juste on redirige vers la page BO de l’objet sans intention particulière ?

Niveau parcours applification, l’idée de ce component redirect c’est qu’on invite l’utilisateur à lier l’objet à une application existante ? Ou bien juste on redirige vers la page BO de l’objet sans intention particulière ?
lguerin commented 2024-03-21 11:41:44 +01:00
Author
Owner
Copy Link

C'est pour, depuis hobo, aller visiter l'objet sur la brique qui va bien et vérifier que c'est bien le bon objet. Ce endpoint est appelé par hobo avant la mise à jour d'une application, l'objet peut exister dans la brique mais pas encore dans hobo, où on n'a pas encore les informations nécessaires pour avoir l'url redirect et l'afficher. Donc le endpoint le renvoie par facilité.

C'est pour, depuis hobo, aller visiter l'objet sur la brique qui va bien et vérifier que c'est bien le bon objet. Ce endpoint est appelé par hobo avant la mise à jour d'une application, l'objet peut exister dans la brique mais pas encore dans hobo, où on n'a pas encore les informations nécessaires pour avoir l'url redirect et l'afficher. Donc le endpoint le renvoie par facilité.
pmarillonnet commented 2024-03-21 11:58:26 +01:00
Owner
Copy Link

Ok, je comprends mieux, merci.

Ok, je comprends mieux, merci.
pmarillonnet marked this conversation as resolved
chrono/apps/export_import/api_views.py Outdated
@ -204,0 +299,4 @@
# no snapshot for this bundle
no_history_elements.append(
{
'type': element['type'],
pmarillonnet commented 2024-03-21 11:06:51 +01:00
Owner
Copy Link

Ok, un peu surpris de la brévité des informations contenues dans ce no_history_elements, par exemple en comparaison des legacy_elements qui présentent un libellé humainement intelligible et une url pour l’objet. J’imagine que l’usage final n’est pas le même.

Ok, un peu surpris de la brévité des informations contenues dans ce `no_history_elements`, par exemple en comparaison des `legacy_elements` qui présentent un libellé humainement intelligible et une url pour l’objet. J’imagine que l’usage final n’est pas le même.
lguerin commented 2024-03-21 11:43:03 +01:00
Author
Owner
Copy Link

Comme dit plus haut, un legacy_element existe dans la brique mais pas encore dans hobo, donc on a besoin de renvoyer des informations permettant à hobo d'afficher un joli message et une url redirect.
Un no_history_element existe forcément dans hobo.

Comme dit plus haut, un legacy_element existe dans la brique mais pas encore dans hobo, donc on a besoin de renvoyer des informations permettant à hobo d'afficher un joli message et une url redirect. Un no_history_element existe forcément dans hobo.
pmarillonnet commented 2024-03-21 11:58:52 +01:00
Owner
Copy Link

Ok, capté, merci.

Ok, capté, merci.
pmarillonnet marked this conversation as resolved
chrono/apps/export_import/api_views.py Outdated
@ -204,0 +315,4 @@
'url': '%s%s?version1=%s&version2=%s'
% (
request.build_absolute_uri('/')[:-1],
reverse(compare_urls[component_type], args=[component.pk]),
pmarillonnet commented 2024-03-21 11:10:24 +01:00
Owner
Copy Link

Pas compris pourquoi on concatène des bouts d’urls au lieu de simplement taper un

request.build_absolute_uri(reverse(compare_urls[…], …))
Pas compris pourquoi on concatène des bouts d’urls au lieu de simplement taper un ``` python request.build_absolute_uri(reverse(compare_urls[…], …)) ```
lguerin commented 2024-03-21 11:50:38 +01:00
Author
Owner
Copy Link

en effet, corrigé

en effet, corrigé
pmarillonnet marked this conversation as resolved
lguerin force-pushed wip/87653-app-bundle-check from e29172ec74 to 1896c33f29 2024-03-21 11:49:47 +01:00 Compare
pmarillonnet approved these changes 2024-03-21 11:59:18 +01:00
lguerin merged commit 1896c33f29 into main 2024-03-21 13:36:29 +01:00
lguerin deleted branch wip/87653-app-bundle-check 2024-03-21 13:36:29 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
pmarillonnet
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/chrono#218
No description provided.
Delete Branch "wip/87653-app-bundle-check"

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?