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

[Metadonnées] Importer dans ce jeux de données => Erreur 404 #2151

Closed
amandine-sahl opened this issue Nov 21, 2022 · 8 comments
Closed

[Metadonnées] Importer dans ce jeux de données => Erreur 404 #2151

amandine-sahl opened this issue Nov 21, 2022 · 8 comments
Labels

Comments

@amandine-sahl
Copy link
Contributor

Version
2.10.3

Description du bug
Lorsque l'on clique sur l'action Importer dans ce jeux de données : Il redirige vers la page : #/import/process/step/1?datasetId=335&resetStepper=true qui n'existe pas

@bouttier
Copy link
Contributor

Actuellement le bouton d’import est hardcodé dans GeoNature, peu importe que le module d’import soit installé ou non. C’est une très mauvaise pratique, et c’est bien pour ça que ce lien est cassé.
Je suis pour dans un premier temps supprimer ce bouton. Et le remettre mais avec un mécanisme permettant au module d’import, lorsqu’il est installé, de venir insérer son bouton dans le module met-adonnées.

@camillemonchicourt
Copy link
Member

OK à voir car le projet DEPOBIO tenait à ce que ce bouton soit proposé depuis le module Métadonnées, mais en effet il ne devrait être présent que si le module Import est installé...

@cecchi-a
Copy link
Contributor

Effectivement de mon côté on me remonte que cette fonctionnalité est nécessaire pour les utilisateurs DEPOBIO

@camillemonchicourt
Copy link
Member

Oui mais elle a été développée de manière non satisfaisante car en dur, même si le module IMPORT n'est pas installé. 🤔

@cecchi-a
Copy link
Contributor

Ce n'est cependant pas une régression introduite par la 2.10 ni la refonte de l'import. Pas de blocage de notre pour la faire évoluer je pense tant que la solution mise en place nous permet un déploiement en début de semaine prochaine ;)

@camillemonchicourt
Copy link
Member

Oui c'était déjà le cas en effet dans les versions précédentes.

@bouttier
Copy link
Contributor

Du coup, la solution sur laquelle on est partie :

@camillemonchicourt
Copy link
Member

Corrigé dans la 2.10.4

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

No branches or pull requests

4 participants