GeotrekCE / Geotrek-rando

Geotrek public trekking website
http://geotrek.fr
27 stars 18 forks source link

updated update process #798

Open pierreloicq opened 3 years ago

pierreloicq commented 3 years ago

(please check correctness)

camillemonchicourt commented 3 years ago

Usually the data folder is outside of the application. And this is recommended. So there is no need to copy data folder at each update.

pierreloicq commented 3 years ago

J'ai essayé en mettant le dossier data en dehors mais ça ne marche pas. Dois-je modifier autre chose que le /etc/nginx/sites-available/geotrek-rando ?

camillemonchicourt commented 3 years ago

Dans settings.custom.json, mon paramètre d'URL de l'API est laissé vide : "API_URL": "", Dans /etc/nginx/sites-available/re, j'ai

[...]
    root /home/myuser/RE/Geotrek-rando/public;
[...]
    location ~ ^/(api|media|static|zip|meta)/ {
        root /home/myuser/RE/data/;
    }
[...]

Mon application et mes dossiers sont dans 2 dossiers séparés au même niveau. Mes données sont accessibles sur https://rando.ecrins-parcnational.fr/api/fr/treks.geojson

pierreloicq commented 3 years ago

Ha oui c'est le "API_URL": "" J'ai édit le commit Merci

gutard commented 3 years ago

A mon avis, il faudrait d'abord supprimer la proposition de mettre "API_URL": "/data" qui est présente quelque part dans la doc. C'est surtout elle qui est le point de départ des soucis. Par contre, je ne vois pas trop l'intérêt de parler de déplacer les données au moment de la mise à jour. Il faut plutôt expliquer comment configurer le tout correctement dès le départ.

camillemonchicourt commented 3 years ago

Oui idem, je pense que cela embrouille un peu ce point qui n'est déjà pas très clair dans la doc, et plutôt à clarifier comme indique @gutard.

pierreloicq commented 3 years ago

La doc mentionne ceci:

URL of the location where geotrek rando should find the api. If you're using Geotrek Admin, it can be either the admin server url or the rando server url (if you activate data sync on the adminside)

ce que je ne comprend pas car je croyais que la synchro était obligatoire (sauf pour la v3 ?). Loupe-je quelque chose ?

Du coup les guillemets vides marchent parce que je n'utilise pas d'URL ?

gutard commented 3 years ago

je croyais que la synchro était obligatoire

C'est surtout pour une question de perfs qu'il n'est pas raisonnable de se passer de la synchro. Se brancher directement sur l'admin est intéressant pour faire des tests, pas pour déployer en prod.

Du coup les guillemets vides marchent parce que je n'utilise pas d'URL ?

Tu utilises toujours une URL pour aller chercher les données de l'API. Ce qui compte c'est que API_URL corresponde avec la configuration nginx. Tu pouvais laisser API_URL=/data mais dans ce cas il fallait adapter la conf nginx pour que l'API soit accessible sur /data ce qui n'était pas le cas.