Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Problème d'import des photos #81

Open
GuillaumeAV opened this issue Jul 6, 2022 · 5 comments
Open

Problème d'import des photos #81

GuillaumeAV opened this issue Jul 6, 2022 · 5 comments
Labels
bug Something isn't working

Comments

@GuillaumeAV
Copy link

Quand je synchronise une donnée CdlT sur archipel.av, les données sont interopérées, c'est trop cool mais pas le logo ... Il faut, a dit @srosset81, utiliser le prédicat "depictedBy" ;)

@GuillaumeAV GuillaumeAV added the bug Something isn't working label Jul 6, 2022
@srosset81
Copy link
Contributor

srosset81 commented Jul 11, 2022

Prévoir une migration de as:image vers pair:depictedBy. Très facile avec migration.replacePredicate qui peut être appelé directement sans code supplémentaire.

https://github.com/assemblee-virtuelle/semapps/blob/next/src/middleware/packages/migration/service.js#L15

@fluidlog
Copy link
Contributor

Suite à notre discussion sur https://forums.assemblee-virtuelle.org/chat/c/archipel-av/34, pour afficher l'organisation des CDLT dans la liste des membres de l'AV sur le site web (Astro), il faut résoudre ce bug.

Si je comprends bien (pour relativiser le terme "facile"), il faut :

  • modifier dans le code d'Archipelago AV / frontend le prédicat dans les fichiers show, edit, list,
  • le déployer sur tous les serveurs actuels, en le documentant comme un "breaking change"
  • utiliser le service de migration pour apporter la modification aussi dans les données de tous les serveurs Archipelago.

J'ai bien compris @srosset81 ?
@mguihal T'es partant pour m'aider à faire ça ? :)

@srosset81
Copy link
Contributor

  • en le documentant comme un "breaking change"

Je n'appelerai pas ça un breaking change. Il faut juste indiquer la commande de migration à lancer (point 3) afin que chacun puisse facilement faire la migration de ses données.

@fluidlog
Copy link
Contributor

J'ai commencé à travailler sur le sujet en local et sur le PAD : https://pad.lescommuns.org/Archipelago-remplacementDesPredicatsPourLesImages
Ensuite, je me suis rendu compte qu'il manquait le prédicat "pair:depictedBy" dans le context.
Or, Seb l'a ajouté récemment dans le middleware 1.09
assemblee-virtuelle/semapps@9ed8aba
Donc nous attendons le passage d'Archipelago en middleware 1.09.

@fluidlog
Copy link
Contributor

@mguihal Rien d'urgent, mais si tu peux inclure le middleware 1.09 dans la prochaine version que tu feras (ou alors moi), on se tient au courant...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants