entrouvert
/
combo
16
1
Fork 0
Code Issues Pull Requests 10 Releases Wiki Activity

API pour l'applification (#60773) #119

Merged
fpeters merged 1 commits from wip/60773-applification into main 2023-10-03 13:40:23 +02:00
Conversation 0 Commits 1 Files Changed 8 +591
fpeters commented 2023-06-19 21:42:20 +02:00
Owner
Copy Link
No description provided.
fpeters force-pushed wip/60773-applification from e53fd466c1 to 529638977a 2023-06-19 21:56:50 +02:00 Compare
fpeters force-pushed wip/60773-applification from 529638977a to 4382faef3a 2023-06-20 07:26:55 +02:00 Compare
fpeters force-pushed wip/60773-applification from 4382faef3a to 50b6d3fb0f 2023-08-08 17:11:41 +02:00 Compare
fpeters force-pushed wip/60773-applification from 50b6d3fb0f to 100a2be39d 2023-08-09 09:10:22 +02:00 Compare
fpeters force-pushed wip/60773-applification from 100a2be39d to e6e5eeb396 2023-08-09 09:14:29 +02:00 Compare
fpeters force-pushed wip/60773-applification from e6e5eeb396 to 30ae285cd1 2023-08-09 17:22:20 +02:00 Compare
fpeters force-pushed wip/60773-applification from 30ae285cd1 to c8a9b315b7 2023-08-10 11:11:15 +02:00 Compare
fpeters force-pushed wip/60773-applification from c8a9b315b7 to b5244ac4e9 2023-08-10 12:47:34 +02:00 Compare
fpeters reviewed 2023-10-02 14:00:22 +02:00
fpeters left a comment
Author
Owner
Copy Link

C'est une version initiale pour démarrer ça et permettre d'être ensuite à plusieurs pour y ajouter les éléments manquants (l'implémentation du bundle-declare, l'affichage dans l'interface de l'application à laquelle appartient une page, la gestion des cartes, etc.).

C'est une version initiale pour démarrer ça et permettre d'être ensuite à plusieurs pour y ajouter les éléments manquants (l'implémentation du bundle-declare, l'affichage dans l'interface de l'application à laquelle appartient une page, la gestion des cartes, etc.).
combo/apps/export_import/api_views.py Outdated
@ -0,0 +63,4 @@
'id': str(page.uuid),
'text': page.title,
'indent': getattr(page, 'level', 0),
'type': 'portal-agent-pages' if is_portal_agent() else 'pages',
fpeters commented 2023-10-02 13:55:22 +02:00
Author
Owner
Copy Link

On distingue deux types de pages pour les applications, selon qu'on est le portail agent ou pas. (et ça devra être la même affaire quand on exportera davantage d'objets type les couches carto).

Ça permet de rester dans le modèle hobo actuel, sans devoir ajouter une notion de type de service qui distinguerait à ce niveau les pages du portail agent des pages du portail usager.

On distingue deux types de pages pour les applications, selon qu'on est le portail agent ou pas. (et ça devra être la même affaire quand on exportera davantage d'objets type les couches carto). Ça permet de rester dans le modèle hobo actuel, sans devoir ajouter une notion de type de service qui distinguerait à ce niveau les pages du portail agent des pages du portail usager.
combo/apps/export_import/api_views.py Outdated
@ -0,0 +136,4 @@
permission_classes = (permissions.IsAuthenticated,)
def put(self, request, *args, **kwargs):
return Response({'err': 0})
fpeters commented 2023-10-02 13:56:14 +02:00
Author
Owner
Copy Link

Je laisse bundle-declare de côté.

Je laisse bundle-declare de côté.
combo/apps/export_import/api_views.py Outdated
@ -0,0 +158,4 @@
json.loads(tar.extractfile(f'{page_type}/{element["slug"]}').read().decode()).get('data')
)
if pages:
import_site({'pages': pages})
fpeters commented 2023-10-02 13:57:00 +02:00
Author
Owner
Copy Link

On prend toutes les pages et on importe tout ça; ça devrait s'étendre facilement pour y ajouter les cartes etc.

On prend toutes les pages et on importe tout ça; ça devrait s'étendre facilement pour y ajouter les cartes etc.
combo/apps/wcs/models.py Outdated
@ -171,0 +178,4 @@
'dependencies': f'{wcs_site_url}api/export-import/forms/{form_slug}/dependencies/',
'redirect': f'{wcs_site_url}api/export-import/forms/{form_slug}/redirect/',
}
yield {'type': 'forms', 'id': form_slug, 'text': self.cached_title, 'urls': urls}
fpeters commented 2023-10-02 13:58:29 +02:00
Author
Owner
Copy Link

Détection des dépendances inter-applications; si on a une cellule démarche on a la démarche en dépendance, et pareil pour les catégories de démarches, et les modèles de fiches.

Détection des dépendances inter-applications; si on a une cellule démarche on a la démarche en dépendance, et pareil pour les catégories de démarches, et les modèles de fiches.
combo/data/models.py Outdated
@ -379,0 +380,4 @@
yield from self.get_children()
yield self.edit_role
yield self.subpages_edit_role
yield from self.groups.all()
fpeters commented 2023-10-02 13:58:56 +02:00
Author
Owner
Copy Link

En dépendances on a aussi les rôles.

En dépendances on a aussi les rôles.
fpeters force-pushed wip/60773-applification from b5244ac4e9 to c2c3d9b893 2023-10-02 14:00:43 +02:00 Compare
fpeters changed title from WIP: API pour l'applification (#60773) to API pour l'applification (#60773) 2023-10-02 19:05:31 +02:00
lguerin reviewed 2023-10-03 09:45:20 +02:00
combo/apps/export_import/api_views.py Outdated
@ -0,0 +67,4 @@
'order': order,
'urls': {
'export': request.build_absolute_uri(
reverse('api-export-import-page-export', kwargs={'pk': page.id})
lguerin commented 2023-10-03 09:33:18 +02:00
Owner
Copy Link

Plutôt utiliser page.uuid partout, non ?

Plutôt utiliser page.uuid partout, non ?
fpeters commented 2023-10-03 10:28:02 +02:00
Author
Owner
Copy Link

Oui en effet, mis dans un commit supplémentaire.

Oui en effet, mis dans un commit supplémentaire.
combo/apps/wcs/models.py Outdated
@ -1020,0 +1051,4 @@
'dependencies': f'{wcs_site_url}api/export-import/cards/{card_slug}/dependencies/',
'redirect': f'{wcs_site_url}api/export-import/cards/{card_slug}/redirect/',
}
yield {'type': 'cards', 'id': card_slug, 'text': self.cached_title, 'urls': urls}
lguerin commented 2023-10-03 09:39:44 +02:00
Owner
Copy Link

Il va manquer, dans le cas d'un cellule avec custo, qui contient un champ de type lien, la dépendance à la page qui peut être paramétrée en target (mais on peut en faire un ticket)

(je pose ça ici, parce que l'app search n'a pas été modifiée dans cette PR: Il manquera aussi, pour les cellules de recherche, les dépendances à des fiches ou pages etc, mais là aussi on peut en faire un ticket)

Il va manquer, dans le cas d'un cellule avec custo, qui contient un champ de type lien, la dépendance à la page qui peut être paramétrée en target (mais on peut en faire un ticket) (je pose ça ici, parce que l'app search n'a pas été modifiée dans cette PR: Il manquera aussi, pour les cellules de recherche, les dépendances à des fiches ou pages etc, mais là aussi on peut en faire un ticket)
fpeters commented 2023-10-03 10:28:33 +02:00
Author
Owner
Copy Link

Ça me va bien de passer par des tickets supplémentaires pour affiner/compléter tout ça (évidemment…).

Ça me va bien de passer par des tickets supplémentaires pour affiner/compléter tout ça (évidemment…).
fpeters force-pushed wip/60773-applification from c2c3d9b893 to 8e66e2e410 2023-10-03 09:58:21 +02:00 Compare
lguerin approved these changes 2023-10-03 10:53:10 +02:00
fpeters force-pushed wip/60773-applification from 414feebf14 to 98b0792864 2023-10-03 11:08:18 +02:00 Compare
fpeters force-pushed wip/60773-applification from 98b0792864 to f5ff197858 2023-10-03 13:11:36 +02:00 Compare
fpeters merged commit f5ff197858 into main 2023-10-03 13:40:23 +02:00
fpeters deleted branch wip/60773-applification 2023-10-03 13:40:23 +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
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/combo#119
No description provided.
Delete Branch "wip/60773-applification"

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?