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

Installation de TaxHub automatique #322

Closed
Adrien-Pajot opened this issue Nov 29, 2021 · 4 comments
Closed

Installation de TaxHub automatique #322

Adrien-Pajot opened this issue Nov 29, 2021 · 4 comments

Comments

@Adrien-Pajot
Copy link

Bonjour,

@mvergez et @Julien-Gr4z ont repris l'installation de GeoNature-citizen dans le cadre d'une prestation avec le PN du Mercantour dans l'optique d'une V1.0. Cette reprise s'est accompagnée d'une réflexion sur les dépendances au schéma taxonomie ainsi qu'à la table l_areas du ref_geo.
Cette installation a été testée et semble maintenant fonctionnelle cf #321. Toutefois, une ligne permet d'installer automatiquement TaxHub au besoin (dont la version est paramétrable).
Maxime, et je partage son avis, trouve que cela crée une dépendance à TaxHub dont l'installation est amenée à évoluer assez régulièrement, or maintenant, nous utilisons seulement son API.

Ne serait-il pas envisageable de renvoyer à la documentation d'installation de TaxHub et que celle-ci soit manuelle ?

Merci pour vos avis,

@samuelpriou
Copy link
Contributor

Bonjour à tous,
Oui il serait finalement plus simple d'installer TaxHub manuellement pour casser la dépendance.

@camillemonchicourt
Copy link
Member

Il y a une dépendance applicative.
On ne peut pas utiliser GeoNature-citizen sans disposer de TaxHub.
C'était donc intéressant de simplifier l'installation de TaxHub pour ceux qui veulent juste GeoNature-citizen, un peu comme on a le install_all dans GeoNature.
Mais si ça complique trop l'installation, tant pis, les administrateurs l'installeront à part manuellement.

@Adrien-Pajot
Copy link
Author

Merci pour ces retours. Sans compliquer l'install je pense que ça permet que les admin comprennent bien ce qu'ils font avec TaxHub, au lieu que ce soit fait automatiquement. On en discute avec @lpofredc et on vous dira ce qu'on aura fait

@camillemonchicourt
Copy link
Member

Depuis la version 1.0.0 on ne gère plus automatiquement l'installation de TaxHub.
A chacun de l'installer manuellement avant si il ne dispose pas déjà de cet outil.

La dépendance au niveau BDD a été grandement réduite pour s'appuyer au maximum sur l'API de TaxHub, mais elle a du être gardée à minima à cause du module de badges qui en dépend.
Évoqué dans #236 (comment).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants