assemblee-virtuelle / archipelago

Fostering interconnections between communities by creating synergies between their platforms
Apache License 2.0
14 stars 6 forks source link

149 les imports de donnees ne marchent plus #156

Closed BastienSig closed 7 months ago

BastienSig commented 7 months ago

J'ai fais un petit fixe, retirer la ligne serveur : @default des index.js d'une ressource permet d'aller chercher dans les dataserver indiqué dans le fichier du même nom.

simonLouvet commented 7 months ago

resolve #149

srosset81 commented 7 months ago

C'est pas un bug, c'est le comportement voulu. De cette manière, on peut faire des recherche multi-serveurs (p.ex. lorsqu'on veut importer une ressource externe ou lorsqu'on veut ajouter un lien vers une ressource externe) sans que, pour autant, l'intégralité des données de tous les data servers soient affichées en liste.

Avec cette proposition, l'instance Archipel de l'AV retournerait l'intégralité des données de la carte presdecheznous, ce qui ne semble pas idéal.

Pour info j'avais proposé que @default soit le comportement par défaut (= ne retourner que les données du serveur par défaut), et qu'on doive explicitement indiquer @all si on veut retourner toutes les données. Voir https://github.com/assemblee-virtuelle/semapps/issues/1053

BastienSig commented 7 months ago

Oui en effet ma formulation de fixe n'était pas adapté, j'essayait de répondre au besoin de guillaume de voir les données des chemins de la transition. Si ce n'est pas adapté on peut supprimer la pr

srosset81 commented 7 months ago

Son besoin était de pouvoir importer les données des CDLT je pense... @GuillaumeAV ? On a jamais listé les données des CDLT directement dans l'instance Archipel de l'AV

EDIT: Effectivement en regardant l'issue https://github.com/assemblee-virtuelle/archipelago/issues/149, Guillaume souhaitait juste savoir pourquoi l'import de l'organisation CDLT (et non toutes les données de l'instance) n'apparaissait plus.

BastienSig commented 7 months ago

Ok j'annule alors !

simonLouvet commented 7 months ago

@GuillaumeAV , @BastienSig t'apelle pour comprendre ton besoin