-
Notifications
You must be signed in to change notification settings - Fork 80
📝 [Documentation] Add command line examples and templates for importing data #4290
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
Conversation
Changements visibles sur cette page : https://geotrek.readthedocs.io/en/mfu-improve-import-data-in-doc/install/import.html#import-pois |
Merci @bruhnild pour cette PR. Une petite erreur dans la commande docker sur l'import des tronçons. Je vais faire une review pour corriger. Sinon ça me semble très bien, plus complet et plus clair ! 🙏 Quelques petits points qui me viennent à l'esprit quand je parcours cette page :
|
docs/install/import.rst
Outdated
|
||
.. code-block:: bash | ||
|
||
sudo geotrek loadpaths \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sudo geotrek loadpaths \ | |
docker compose run --rm web ./manage.py loadpaths \ |
Oui pour moi il vaut mieux enlever les parties liées au module Plongée. Et en effet, je privilégierai d'indiquer que les commandes classiques et ceux en Docker savent comment les adapter avec la doc Docker qui le mentionne. |
Merci pour vos retours. Je suis en accord avec la proposition de retirer les éléments liés au module Plongée, et j'ai effectué les modifications nécessaires. |
@bruhnild oui pourquoi pas laisser un exemple dans un encart dédié si tu penses que c'est mieux et plus clair comme ça |
Description
Related Issue
Checklist