entrouvert
/
passerelle
15
0
Fork 0
Code Issues Pull Requests 15 Releases Activity

astech: add endpoint to get view data (#82416) #366

Merged
smihai merged 3 commits from wip/82416-astech-add-endpoints-to-get-views-data into main 2023-10-17 16:52:49 +02:00
Conversation 0 Commits 3 Files Changed 2 +147
smihai commented 2023-10-16 14:58:40 +02:00
Owner
Copy Link
No description provided.
smihai force-pushed wip/82416-astech-add-endpoints-to-get-views-data from a54dafa973 to aa2d68b58e 2023-10-16 15:21:37 +02:00 Compare
tnoel requested changes 2023-10-16 15:43:21 +02:00
passerelle/apps/astech/models.py Outdated
@ -443,0 +487,4 @@
)
def get_view_data(self, request, code):
endpoint = 'apicli/data/%s/results' % code
results = self.call(endpoint, json={'data': {'filters': []}})
tnoel commented 2023-10-16 15:43:15 +02:00
Owner
Copy Link

Tu n'auras pas besoin de gérer "q" et "id" ici, et de construire un filtre si l'un ou l'autre est présent ?

Tu n'auras pas besoin de gérer "q" et "id" ici, et de construire un filtre si l'un ou l'autre est présent ?
smihai commented 2023-10-17 14:56:12 +02:00
Author
Owner
Copy Link

Avec le paramètre datasource=True ça fonctionne tout seul.

Avec le paramètre `datasource=True` ça fonctionne tout seul.
smihai marked this conversation as resolved
passerelle/apps/astech/models.py Outdated
@ -443,0 +489,4 @@
endpoint = 'apicli/data/%s/results' % code
results = self.call(endpoint, json={'data': {'filters': []}})
for result in results:
result['id'] = result['BIEN_ID']
tnoel commented 2023-10-16 15:43:17 +02:00
Owner
Copy Link

À mon avis ça ne marche que pour le code ASTECH_BIENS, à voir avec les autres (ASTECH_MESURES, etc).

À mon avis ça ne marche que pour le code ASTECH_BIENS, à voir avec les autres (ASTECH_MESURES, etc).
smihai commented 2023-10-17 14:56:52 +02:00
Author
Owner
Copy Link

En effet.
Et je demande pour ça 2 paramètres pour spécifier les colonnes contenant l'id et le texte.

En effet. Et je demande pour ça 2 paramètres pour spécifier les colonnes contenant l'id et le texte.
smihai marked this conversation as resolved
smihai force-pushed wip/82416-astech-add-endpoints-to-get-views-data from aa2d68b58e to b34e68a062 2023-10-17 14:56:11 +02:00 Compare
smihai requested review from tnoel 2023-10-17 14:57:01 +02:00
tnoel approved these changes 2023-10-17 16:29:51 +02:00
tnoel left a comment
Owner
Copy Link

J'ai juste un commentaire sur text_column à la place de label_column, mais rien qui t'oblige.

J'ai juste un commentaire sur text_column à la place de label_column, mais rien qui t'oblige.
passerelle/apps/astech/models.py Outdated
@ -443,0 +488,4 @@
'example_value': 'ASTECH_BIENS',
},
'id_column': {'description': _('Name of column contaning the id'), 'example_value': 'BIEN_ID'},
'label_column': {
tnoel commented 2023-10-17 16:25:38 +02:00
Owner
Copy Link

J'aurai nommé ça "text_column" mais ça roule ainsi.

(j'imagine aussi qu'un classique "text_template" sera plus tard demandé, mais on peut attendre le «cas d'usage» et un ticket correspondant)

J'aurai nommé ça "text_column" mais ça roule ainsi. (j'imagine aussi qu'un classique "text_template" sera plus tard demandé, mais on peut attendre le «cas d'usage» et un ticket correspondant)
smihai commented 2023-10-17 16:52:37 +02:00
Author
Owner
Copy Link

J'aurai nommé ça "text_column" mais ça roule ainsi.

Yep, c'est fait.

> J'aurai nommé ça "text_column" mais ça roule ainsi. > Yep, c'est fait.
smihai marked this conversation as resolved
smihai force-pushed wip/82416-astech-add-endpoints-to-get-views-data from b34e68a062 to 4789f1e1ff 2023-10-17 16:41:57 +02:00 Compare
smihai merged commit 4789f1e1ff into main 2023-10-17 16:52:49 +02:00
smihai deleted branch wip/82416-astech-add-endpoints-to-get-views-data 2023-10-17 16:52:49 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
tnoel
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/passerelle#366
No description provided.
Delete Branch "wip/82416-astech-add-endpoints-to-get-views-data"

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?