Closed srosset81 closed 4 months ago
Hello,
Je remarque que dans cette PR il y a plusieurs références à l'Assemblée Virtuelle dans le code (le fichier middleware/imports/export.xml
notamment, ainsi que le fichier wordpress.service.js
).
Si c'est un développement spécifique pour l'instance de l'Archipelago de l'Assemblée Virtuelle, je suggère de ne pas faire les modifications sur ce repo, mais sur https://github.com/assemblee-virtuelle/deployment_semapps_av, afin de laisser celui-là le plus générique possible.
@mguihal Merci, bonne remarque. Il y a aussi quelques améliorations (upgrade SemApps notamment), mais je vais extraire le service d'importation.
Je pense qu'il ne faut pas hésiter à faire une nouvelle release après un merge comme ça (si tout est fonctionnel)