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

general: import plugin system (#82198) #161

Merged
ecazenave merged 1 commits from wip/82198-api-export-import into main 2023-10-17 12:34:52 +02:00
Conversation 2 Commits 1 Files Changed 10 +798
ecazenave commented 2023-10-10 16:54:25 +02:00
Owner
Copy Link
No description provided.
ecazenave added 1 commit 2023-10-10 16:54:26 +02:00
gitea/chrono/pipeline/head There was a failure building this commit Details
bb50cc4270 general: import plugin system (#82198)
fpeters commented 2023-10-10 16:59:52 +02:00
Owner
Copy Link

On pourrait juste avoir des déclarations statiques des URL etc., ne pas intégrer toute cette partie dynamique, je trouve.

On pourrait juste avoir des déclarations statiques des URL etc., ne pas intégrer toute cette partie dynamique, je trouve.
ecazenave force-pushed wip/82198-api-export-import from bb50cc4270 to 750dd001c7 2023-10-10 17:04:18 +02:00 Compare
ecazenave force-pushed wip/82198-api-export-import from 750dd001c7 to 25d83065db 2023-10-10 18:14:42 +02:00 Compare
ecazenave force-pushed wip/82198-api-export-import from 25d83065db to 26fae21829 2023-10-12 15:31:52 +02:00 Compare
ecazenave force-pushed wip/82198-api-export-import from 26fae21829 to a7e6724f6c 2023-10-12 18:19:53 +02:00 Compare
ecazenave force-pushed wip/82198-api-export-import from a7e6724f6c to f75e3791b9 2023-10-13 16:12:31 +02:00 Compare
ecazenave force-pushed wip/82198-api-export-import from f75e3791b9 to bcd2bc75e5 2023-10-13 16:41:00 +02:00 Compare
ecazenave force-pushed wip/82198-api-export-import from bcd2bc75e5 to 8f8c2e3841 2023-10-13 16:57:31 +02:00 Compare
ecazenave force-pushed wip/82198-api-export-import from 8f8c2e3841 to f5e81c0786 2023-10-13 17:11:25 +02:00 Compare
ecazenave commented 2023-10-13 17:22:30 +02:00
Author
Owner
Copy Link

On pourrait juste avoir des déclarations statiques des URL etc., ne pas intégrer toute cette partie dynamique, je trouve.

Viré.

Pour le reste, à grands coups de copier/coller/modifier de l'API de combo.

Pour les dépendances d'une agenda je ne vois rien d'autre que les rôles et les agendas rdv dans le cas d'un agenda virtuel (mais je loupe peut-être des choses, je ne suis pas particulièrement à jour sur chrono).

Hésitation sur le fait d'inclure dans les dépendances les calendriers d’indisponibilités et autres exceptions mais ça ne parait pas relever d'une application.

> On pourrait juste avoir des déclarations statiques des URL etc., ne pas intégrer toute cette partie dynamique, je trouve. Viré. Pour le reste, à grands coups de copier/coller/modifier de l'API de combo. Pour les dépendances d'une agenda je ne vois rien d'autre que les rôles et les agendas rdv dans le cas d'un agenda virtuel (mais je loupe peut-être des choses, je ne suis pas particulièrement à jour sur chrono). Hésitation sur le fait d'inclure dans les dépendances les calendriers d’indisponibilités et autres exceptions mais ça ne parait pas relever d'une application.
ecazenave changed title from WIP: general: import plugin system (#82198) to general: import plugin system (#82198) 2023-10-13 17:22:37 +02:00
lguerin reviewed 2023-10-14 12:52:55 +02:00
chrono/apps/export_import/api_views.py Outdated
@ -0,0 +125,4 @@
return redirect(reverse('chrono-manager-agenda-view', kwargs={'pk': agenda.pk}))
class BundleCheck(GenericAPIView):
lguerin commented 2023-10-14 12:51:07 +02:00
Owner
Copy Link

il n'est pas nécessaire d'implémenter ce endpoint tant qu'on n'a pas d'historisation dans chrono. On peut juste renvoyer tout le temps {'data': {}}) et ça suffit pour hobo dans un premier temps.
(et c'est même mieux, sinon tant qu'on n'aura pas d'historisation on aura tout le temps un écran de warning côté hobo qui dit "pas d'historique")

il n'est pas nécessaire d'implémenter ce endpoint tant qu'on n'a pas d'historisation dans chrono. On peut juste renvoyer tout le temps `{'data': {}})` et ça suffit pour hobo dans un premier temps. (et c'est même mieux, sinon tant qu'on n'aura pas d'historisation on aura tout le temps un écran de warning côté hobo qui dit "pas d'historique")
ecazenave commented 2023-10-17 10:41:52 +02:00
Author
Owner
Copy Link

ok, fait.

ok, fait.
lguerin reviewed 2023-10-17 09:26:04 +02:00
chrono/apps/export_import/urls.py Outdated
@ -0,0 +39,4 @@
path('export-import/bundle-check/', api_views.bundle_check),
path('export-import/bundle-declare/', api_views.bundle_declare),
path('export-import/bundle-import/', api_views.bundle_import),
path('export-import/bundle-unlink/', api_views.bundle_unlink),
lguerin commented 2023-10-17 09:26:00 +02:00
Owner
Copy Link

erreur dans combo (que je vais corriger), l'url c'est 'export-import/unlink/'

erreur dans combo (que je vais corriger), l'url c'est `'export-import/unlink/'`
ecazenave commented 2023-10-17 10:42:02 +02:00
Author
Owner
Copy Link

Fait.

Fait.
ecazenave force-pushed wip/82198-api-export-import from f5e81c0786 to cba5520541 2023-10-17 10:29:57 +02:00 Compare
ecazenave requested review from lguerin 2023-10-17 10:42:22 +02:00
lguerin approved these changes 2023-10-17 11:50:47 +02:00
ecazenave merged commit cba5520541 into main 2023-10-17 12:34:52 +02:00
ecazenave deleted branch wip/82198-api-export-import 2023-10-17 12:34:52 +02: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/chrono#161
No description provided.
Delete Branch "wip/82198-api-export-import"

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?