-
Notifications
You must be signed in to change notification settings - Fork 14
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
Publish docs on readthedocs #1254
Conversation
Changements visibles ici : https://geotrek-rando-v3.readthedocs.io/latest/ |
OK pour moi (hormis le copyright), ça rend bien comme ça. |
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.
Quelques ajustements.
Par contre, j'avais identifié quelques soucis avec mkdocs. Exemple ici de code bloc indenté : La seule bonne solution pour cela semble être l'ajout de l'extension https://facelessuser.github.io/pymdown-extensions/extensions/superfences/, comme discuté ici : mkdocs/mkdocs#282 (comment) Et les notes, warning and co qui ne sont pas interprétés : Je ne les ai peut-être pas renseignés comme il faut. Je creuse un peu ça. Après si |
Ok pour merger ? |
e3cdfcc
to
5fa6e94
Compare
Nouveau thème Material for mkdocs ==> https://geotrek-rando-v3.readthedocs.io/latest/ |
7819c76
to
af65b7f
Compare
Quelques sujets / questions :
|
|
Pour le dernier point, aucun apport de doc n'a été fait depuis le fork de cette PR. Juste le changelog qui a conflict avec la release de ce matin |
J'ai retiré les mentions Rando V3 et précisé J'ai voulu mettre en place un mécanisme de changement de langue (anglais/français) mais :
|
@bruhnild pour moi pas besoin d'activer deux langues pour le moment. Si tu mentionné quand on mettais le lien que la page était français c'est suffisant. Merci pour la mise à jour de la doc. 🙏 Je n'ai pas vérifié (pas assez de réseau pour charger la page de diff), mais il faut aussi en profiter pour mettre à jour le readme du projet et indiquer la lien vers la nouvelle doc. Pour le reste c'est ok pour moi, on peut merger. Rien à voir, mais est-ce qu'on ne profiterai pas de cette "nouvelle" doc pour en faire la com' auprès de la communauté ? Un mail sur la mailing liste en mode "Heyyy, pour info voici la nouvelle doc GTR3 : . N'hésitez pas à aller la voir et à nous remonter sur ce ticket (insérer lien vers un ticket) si vous trouvez certaines choses manquantes ou peu claires, nous avons besoin de vos retours pour l’améliorer". Idem si vous avez des idées et souhaits de nouvelles fonctionnalités ou si vous identifiez des choses qui vous paraissent manquantes" ? Je me dis que ça pourrait être l'occasion d'obtenir des retours de la communauté car peut-être certaines personnes n'osent pas ou n'y pensent pas le reste du temps ? |
@dtrucs peux-tu regarder le diff et voir si ça te semble ok de ton côté les modifs avant de merger ? |
Readme mis à jour. |
@bruhnild |
colorama~=0.4 | ||
paginate~=0.5 | ||
regex>=2022.4 | ||
requests~=2.26 |
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.
@submarcos could you review parts related to python stuff?
|
186c5cb
to
b9d1085
Compare
b9d1085
to
9094631
Compare
Effectivement, il y a eu un commit malheureux qui a embarqué des fichiers indésirables. Le dossier |
Le fichier README.md sera aussi celui affiché sur la page d'accueil quand on arrive sur le Github : https://github.com/GeotrekCE/Geotrek-rando-v3 On ne devrait pas inclure ce README dans la documentation, et faire en sorte qu'il contienne le minimum nécessaire quand on arrive sur le Github du projet (image et texte de présentation de base, lien vers la doc...). |
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.
Très chouette cette nouvelle page de readme.
Maintenant j'ai envie de faire la même chose sur GTA 😆
README.md
Outdated
|
||
Geotrek-rando is the public web application displaying the interface you can use to value your territory treks and tourism products! | ||
Demo available at https://gtr3demo.ecrins-parcnational.fr. | ||
Demo available at [https://gtr3demo.ecrins-parcnational.fr](https://gtr3demo.ecrins-parcnational.fr). |
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.
Demo available at [https://gtr3demo.ecrins-parcnational.fr](https://gtr3demo.ecrins-parcnational.fr). | |
You can find two demonstration website at the following addresses : | |
- [https://gtr3demo.ecrins-parcnational.fr](https://gtr3demo.ecrins-parcnational.fr) | |
- [https://demo-rando.geotrek.fr/](https://demo-rando.geotrek.fr) | |
Discover more users close to your place by going onto the [user map](https://geotrek.fr/utilisateurs.html). |
Je me dis que pour les deux instances de démo ça serait chouette aussi de mettre un lien vers l'admin (+ identifiants) pour quel les curieu·x·ses puissent aller tester des modifications ? Mais est-ce l'endroit pour mettre des liens vers l'admin ? Est-ce qu'il ne faudrait pas plutôt mettre un lien vers une section "Démo" équivalente sur le répo de Geotrek-Admin (à créer) ?
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.
On pourrait aussi mettre directement 2 ou 3 exemples de vrais portails pour montrer des vrais usages, moins basiques, non ?
Mettre les liens des admins, etc... pourquoi pas, mais faut pas que ça devienne un peu complexe de s'y retrouver dans trop de liens et d'infos. Pas d'avis tranché donc.
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.
J'ai créé une section "Examples of Geotrek-rando portals" avec 4 randos + un montage d'images pour illustrer.
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.
pour les liens de l'admin je n'ai rien fait pour l'instant, pas trop d'idée sur la façon de les présenter dans le readme.
@dtrucs go pour merger cette PR ? |
No description provided.