Skip to content
This repository has been archived by the owner on Jul 28, 2023. It is now read-only.

updated update process #798

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

pierreloicq
Copy link
Contributor

(please check correctness)

@camillemonchicourt
Copy link
Member

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
Copy link
Contributor Author

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
Copy link
Member

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
Copy link
Contributor Author

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

@gutard
Copy link
Contributor

gutard commented Nov 9, 2020

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
Copy link
Member

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
Copy link
Contributor Author

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
Copy link
Contributor

gutard commented Nov 10, 2020

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.

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

Successfully merging this pull request may close these issues.

3 participants