betagouv / eac-api

API de la plateforme EAC. Distribue des données sur les Acteurs Culturels et leurs actions.
https://api.education-artistique-culturelle.fr/
MIT License
2 stars 1 forks source link

Versions #27

Closed rap2hpoutre closed 5 years ago

rap2hpoutre commented 5 years ago

Ajout d'une classe Version et d'une fonction version qui fonctionne de la manière suivante :

version('Actor', one_actor_or_many_actors)

@vinyll Cette PR est surtout l'occasion de discuter de l'implémentation, le mieux serait d'en parler en direct dans la journée. Les principaux sujets de discussion selon moi (il y en a surement d'autres) :

À ta dispo, dès que possible pour qu'on passe à autre chose.

rap2hpoutre commented 5 years ago

Fix #9

rap2hpoutre commented 5 years ago

Ok, à la réflexion, je vais me baser sur ce commentaire :

En même temps, si ca marche, pourquoi pas le prendre et le challenger ensuite (puisque c'est une amélioration fonctionnelle qui répond à un besoin)

J'ai envie d'avancer et la discussion a eu lieu par commentaires. J'ai expérimenté l'approche que tu souhaitais (différente des deux que j'avais identifiées et proposées), la seule autre alternative maintenant serait que tu la fasse toi-même, mais on retomberait dans quelque chose que j'aurai probablement "du mal à m'approprier" (du fait que l'approche n'est pas celle que j'identifiais). Concernant ce problème d'appropriation des routes liées aux création/mises à jours d'acteurs, je crois que le problème est partiellement indépendant de cette PR et antérieur : la fonction de createOrUpdateActor est de toute façon pas terrible, on l'avait identifié, car elle veut factoriser du code qui n'a peut être pas besoin de l'être (en tout cas pas forcément de cette manière là).