cozy / cozy-client

Document store and React components for the Cozy platform
MIT License
13 stars 13 forks source link

2 routes pour le doctype io.cozy.konnectors #415

Open kosssi opened 5 years ago

kosssi commented 5 years ago

Je crée cette issue pour avoir un historique autour de ce doctype.

Je ne suis pas sur de comprendre pourquoi il y a 2 routes pour le même doctype : /data/io.cozy.konnectors et /konnectors. Si j'ai bien compris c'est seulement pour ajouter des links dans le retour de la requête.

Historiquement cozy-client utilisait /data/io.cozy.konnectors ce qui permettait de faire de la synchronisation, de la recherche facilement. Nous avons fait un changement pour utiliser:

Si nous pouvions repasser tout sur /data/io.cozy.konnectors et que ça soit cozy-client qui ajoute les links ça simplifierai son utilisation j'ai l'impression.

Voir le code

gregorylegarec commented 5 years ago

Ça implique des risques de régression au moins côté Home et Store, donc complètement pour migrer mais il faudra faire ça avec attention et sérénité.

nono commented 5 years ago

Je ne suis pas sur de comprendre pourquoi il y a 2 routes pour le même doctype : /data/io.cozy.konnectors et /konnectors.

Côté stack, /data/io.cozy.konnectors n'est disponible qu'en lecture. Pour les écritures, il faut passer par les routes de /konnectors. Ces routes exposent du JSON-API contrairement aux routes de /data qui exposent le format de CouchDB. On avait ajouté une route de lecture (en fait, 2, une pour lister et une pour une app précise) sur /konnectors, avec du JSON-API, pour simplifier les développements front (ie qu'ils puissent manipuler que du JSON-API pour les appels sur les connecteurs). Aujourd'hui, on préfère passer par /data pour le offline-first, et c'est donc logique de vouloir changer de routes.