entrouvert
/
hobo
16
0
Fork 0
Code Issues Pull Requests 16 Releases Activity

appeler wcs pour déclarer une app ou supprimer les liens d'une app (#74659) #20

Merged
lguerin merged 2 commits from wip/74659-application-links into main 2023-04-17 16:17:19 +02:00
Conversation 2 Commits 2 Files Changed 3 +102 -19
lguerin commented 2023-02-20 16:35:15 +01:00
Owner
Copy Link
No description provided.
lguerin added 2 commits 2023-02-20 16:35:15 +01:00
gitea/hobo/pipeline/pr-main Something is wrong with the build of this commit Details
gitea/hobo/pipeline/head There was a failure building this commit Details
f0b56537aa
applications: unlink app in services on deletion (#74659)
lguerin changed title from wip/74659-application-links to appeler wcs pour déclarer une app ou supprimer les liens d'une app (#74659) 2023-02-20 16:35:45 +01:00
lguerin force-pushed wip/74659-application-links from f0b56537aa to 279e09fbea 2023-02-21 19:21:05 +01:00 Compare
lguerin force-pushed wip/74659-application-links from 279e09fbea to a699f97053 2023-02-22 16:42:23 +01:00 Compare
aberriot reviewed 2023-03-02 10:53:29 +01:00
aberriot left a comment
Owner
Copy Link

Je ne suis pas encore assez à l'aise pour relire entrouvert/wcs#102, mais j'ai testé celle ci en local, et le comportement me semble bon : ça appelle bien l'URL wcs pour le unlink lors d'une suppression d'application dans hobo. (ça plante chez moi parce que 404 côté wcs puisque je suis sur wcs@main, mais le error handling est bon et le code s'exécute).

Sur le fond, j'ai une question liée au comportement de l'unlink/suppression: dans l'interface, on clique sur « Application -> Supprimer ». Je m'attends à ce que cela supprime l'application et les ressources associées, notamment les formulaires, worfklows et catégories. Or, si j'ai bien compris le comportement est inverse : les ressources sont conservées, mais déliées de l'application, via l'appel à POST /unlink/.

Est-ce que cela ne risque pas de poser problème pour le use case d'un admin qui souhaiterai tester une application sur son instance, et se retrouve dans l'incapacité de la désinstaller proprement (l'application laisse trainer son état derrière elle) ?

Je ne suis pas encore assez à l'aise pour relire https://git.entrouvert.org/entrouvert/wcs/pulls/102, mais j'ai testé celle ci en local, et le comportement me semble bon : ça appelle bien l'URL wcs pour le unlink lors d'une suppression d'application dans hobo. (ça plante chez moi parce que 404 côté wcs puisque je suis sur wcs@main, mais le error handling est bon et le code s'exécute). Sur le fond, j'ai une question liée au comportement de l'unlink/suppression: dans l'interface, on clique sur « Application -> Supprimer ». Je m'attends à ce que cela supprime l'application et les ressources associées, notamment les formulaires, worfklows et catégories. Or, si j'ai bien compris le comportement est inverse : les ressources sont conservées, mais déliées de l'application, via l'appel à POST /unlink/. Est-ce que cela ne risque pas de poser problème pour le use case d'un admin qui souhaiterai tester une application sur son instance, et se retrouve dans l'incapacité de la désinstaller proprement (l'application laisse trainer son état derrière elle) ?
lguerin commented 2023-03-02 10:56:28 +01:00
Author
Owner
Copy Link

Je m'attends à ce que cela supprime l'application et les ressources associées, notamment les formulaires, worfklows et catégories

Il y a une popup de confirmation qui explique que les objets ne sont pas supprimés dans les services il me semble

Le bouton supprimer existait déjà, je n'ai fait que rajouter l'appel unlink

> Je m'attends à ce que cela supprime l'application et les ressources associées, notamment les formulaires, worfklows et catégories Il y a une popup de confirmation qui explique que les objets ne sont pas supprimés dans les services il me semble Le bouton supprimer existait déjà, je n'ai fait que rajouter l'appel unlink
aberriot commented 2023-03-02 11:16:11 +01:00
Owner
Copy Link

Il y a une popup de confirmation qui explique que les objets ne sont pas supprimés dans les services il me semble

Le bouton supprimer existait déjà, je n'ai fait que rajouter l'appel unlink

C'est bien noté dans la popup de confirmation oui, ma remarque portait plutôt sur l'impossibilité fonctionnelle de supprimer réellement une application, mais ça sort probablement du scope de la PR :)

> Il y a une popup de confirmation qui explique que les objets ne sont pas supprimés dans les services il me semble > > Le bouton supprimer existait déjà, je n'ai fait que rajouter l'appel unlink C'est bien noté dans la popup de confirmation oui, ma remarque portait plutôt sur l'impossibilité fonctionnelle de supprimer réellement une application, mais ça sort probablement du scope de la PR :)
aberriot approved these changes 2023-03-02 11:16:31 +01:00
lguerin added 2 commits 2023-03-28 13:19:30 +02:00
lguerin added 2 commits 2023-04-17 15:26:27 +02:00
lguerin merged commit a1e8c69eef into main 2023-04-17 16:17:19 +02:00
lguerin deleted branch wip/74659-application-links 2023-04-17 16:17:19 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
aberriot
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/hobo#20
No description provided.
Delete Branch "wip/74659-application-links"

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?