Affichage des responsables de la donnée #4230
Conversation
| :legal_owners_aom, | ||
| :legal_owners_region, |
There was a problem hiding this comment.
Avec ce preload je pense que tu récupères tous les champs géographiques des AOMs et des régions (colonne geom). Ça peut commencer à faire beaucoup à sortir de la base de données pour les agrégats.
Voir si on peut restreindre les champs qui sont preload ?
| "aoms" => legal_owners_aom(dataset.legal_owners_aom), | ||
| "regions" => legal_owners_region(dataset.legal_owners_region), |
There was a problem hiding this comment.
Les régions sont des AOMs. Notre retour d'API expose notre modèle de données qui ne correspond peut-être pas à la réalité. Est-ce qu'on souhaite regrouper ?
There was a problem hiding this comment.
Ahah, c’est une bonne question. À minima il faut pour moi les différencier – c’est important si c’est un AOM local, un AOM local délégué à la région, ou un AOM régional, c’est pas les mêmes prérogatives, mais ça pourrait aussi être au sein d’un array avec un type d’objet indiqué. J’y réfléchis. Poke @cyrilmorin
Je sais pas si on a des utilisateurs de l’API à qui on peut poser la question ? Peut-être déjà l’ART puisqu’ils scriptent autour de notre API.
There was a problem hiding this comment.
J’aurais tendance à dire qu’on laisse comme ça, et qu’on fait un appel à participation pour définir notre API plus pérenne ?
apps/transport/lib/transport_web/templates/dataset/details.html.heex
Outdated
Show resolved
Hide resolved
apps/transport/lib/transport_web/templates/dataset/details.html.heex
Outdated
Show resolved
Hide resolved
apps/transport/lib/transport_web/templates/dataset/details.html.heex
Outdated
Show resolved
Hide resolved
Co-authored-by: Antoine Augusti <antoine.augusti@transport.data.gouv.fr>
J’ai rajouté les responsables légaux dans la barre latérale de fiche d’identité du dataset, ainsi que dans l’API.
Pour la partie graphique, c’est vu avec @cyrilmorin, notamment le wording et le comportement (liens vers les pages d’AOM et de Région), je vous ai mis une capture d’écran plus bas.
Pour l’API, je suis preneur de retours, j’ai peu d’expérience en design d’API. Les responsables légaux peuvent être un ensemble de régions, d’AOM locaux ou une entreprise, les 3 possibilités pouvant se combiner (voir exemple plus bas, où il y a à la fois une région et des AOM locaux). J’ai choisi de faire une structure qui est toujours là, avec des arrays qui peuvent être vides pour les régions et les AOM, et un champ nullable pour le SIREN d’entreprise.
Petites bricoles au passage :
DB.Datasetqui ne servaient plus, ça dégage.covered_areaet auxlegal_owners. Les objets AOM et Région sont identiques ainsi entre la covered area, qui était déjà là, et les legal owners, qui est nouveau. Par contre la structure englobante est un peu différente.La sidebar devient donc :
Et l’API donne ça (je laisse les covered_area et legal_owners pour comparaison) :
{ […] "covered_area": { "name": "Nouvelle-Aquitaine", "region": { "insee": "75", "name": "Nouvelle-Aquitaine" }, "type": "region" }, […] "id": "6246416dbd6bdfe8bceb4ce1", "legal_owners": { "aoms": [ { "name": "CU Limoges Métropole", "siren": "248719312" }, { "name": "CA du Bassin de Brive", "siren": "200043172" }, { "name": "Bordeaux Métropole", "siren": "243300316" }, { "name": "Syndicat des mobilités Pays Basque - Adour", "siren": "256401605" }, { "name": "CA du Grand Angoulême", "siren": "200071827" }, { "name": "CA Tulle Agglo", "siren": "241927201" }, { "name": "CA Mont de Marsan Agglomération", "siren": "244000808" }, { "name": "CA du Grand Dax", "siren": "244000675" }, { "name": "CA Grand Châtellerault", "siren": "248600413" }, { "name": "CA de Saintes", "siren": "200036473" }, { "name": "CA du Libournais", "siren": "200070092" }, { "name": "CA de La Rochelle", "siren": "241700434" }, { "name": "CU du Grand Poitiers", "siren": "200069854" }, { "name": "CA du Niortais", "siren": "200041317" }, { "name": "CA Le Grand Périgueux", "siren": "200040392" }, { "name": "CA Bergeracoise", "siren": "200070647" }, { "name": "CA Val de Garonne Agglomération", "siren": "200030674" }, { "name": "CA du Grand Cognac", "siren": "200070514" }, { "name": "CA Rochefort Océan", "siren": "200041762" }, { "name": "CA Royan Atlantique", "siren": "241700640" }, { "name": "CC Maremne Adour Côte Sud", "siren": "244000865" }, { "name": "CA du Bocage Bressuirais", "siren": "200040244" }, { "name": "CA du Grand Guéret", "siren": "200034825" } ], "company": null, "regions": [ { "insee": "75", "name": "Nouvelle-Aquitaine" } ] }, […] }Fixes #3717