Espace réutilisateur : formulaire pour ajouter des données améliorées #4437
Conversation
AntoineAugusti
changed the title
| defp existing_improved_data(%DB.Dataset{id: dataset_id}, [%DB.Organization{id: organization_id}]) do | ||
| DB.ReuserImprovedData | ||
| |> where([r], r.dataset_id == ^dataset_id and r.organization_id == ^organization_id) | ||
| |> DB.Repo.one() | ||
| end |
There was a problem hiding this comment.
Une ressource améliorée est déposée au nom de l'organisation éligible au pilote, ce n'est pas scopé par le contact qui a fait l'action (même si le contact est bien enregistré)
There was a problem hiding this comment.
C'est hors scope mais je corrige la Content Security Policy. En local les images des organisations/utilisateurs n'étaient pas affichés car le domaine demo-static.data.gouv.fr n'était pas autorisé
|
|
||
| def change do | ||
| create table(:reuser_improved_data) do | ||
| add(:dataset_id, references(:dataset, on_delete: :delete_all), null: false) |
There was a problem hiding this comment.
On pourrait retrouver le dataset_id avec la ressource mais je pense que ça facilitera bien le travail de le stocker ici.
|
|
||
| config :transport, | ||
| data_sharing_pilot_dataset_custom_tag: "repartage_donnees", | ||
| data_sharing_pilot_eligible_datagouv_organization_ids: [ |
There was a problem hiding this comment.
On met notre organisation en tant qu'admins + les calculateurs d'itinéraires qui ont déjà une organisation existante
C'est déjà pas mal |
| |> assoc_constraint(:resource) | ||
| |> assoc_constraint(:contact) | ||
| |> assoc_constraint(:organization) | ||
| |> unique_constraint([:dataset_id, :organization_id]) |
There was a problem hiding this comment.
Je m'attendais à voir le resource_id membre de la clef d'unicité. Un réutilisateur ne fournit-il pas une version améliorée d'une ressource ?
En lien avec #4011
Cette PR ajoute un formulaire dans l'espace réutilisateur, permettant de partager des données améliorées.
Ce formulaire est visible uniquement :
public-transitavec le 🏷️repartage_donnees(ajouté par @cyrilmorin pour les JDDs dans le périmètre de l'expérimentation)L'utilisateur peut alors choisir de partager une URL vers une ressource améliorée, il doit sélectionner le GTFS correspondant. Ce partage de données est fait au nom de l'organisation éligible : si un contact a déjà repartagé quelque chose, il ne sera pas possible pour un collègue de le faire.
Ce qui n'est pas fait
Cette PR se limite à créer une nouvelle table, le modèle Ecto, une logique d'éligibilité, un formulaire pour enregistrer une URL en base de données.
Il n'y a pas de snapshot de la ressource améliorée partagée, impossible d'éditer/supprimer ce fichier partagé depuis une interface, la liste des organisations éligible est figée, les JDDs qui sont dans le pilote doivent être ajoutés en favoris ❤️ par les contacts auparavant.
Démo
Screen.Recording.2025-02-04.at.10.55.09.mov