Update Docker Compose Dev Environment (#580) #584
Conversation
| volumes: | ||
| meilisearch: | ||
| data: |
There was a problem hiding this comment.
Why though? Sounds easier to not have to mount a manual data dir?
There was a problem hiding this comment.
This is mostly up to preference, I would much rather prefer having the /data directory to be persistent so that I can modify the sqlite db in case it is needed
|
|
||
| - Clone the repo and navigate to the root of the repo. | ||
| - Run `cp /docker/.env.sample /docker/.env` | ||
| - Make the necessary changes to the .env file (setting the DOCKER_DATA_DIR, NEXTAUTH_SECRET, and MEILI_MASTER_KEY) |
There was a problem hiding this comment.
You don't really need to set nextauth_secret or even meili key during development. So I wouldn't worry too much about them
There was a problem hiding this comment.
Wouldn't it be a good idea to set it up anyways, just to make sure that any development does not break functionality offered by nextauth or meilisearch?
There was a problem hiding this comment.
I am too prefer having nextauth and meilisearch being ready while developing
There was a problem hiding this comment.
Just to be clear. They will be ready and usable. The requirements of the variables being set are only when deployed to prod.
There was a problem hiding this comment.
oh gotcha because you want that stuff seeded with a random value
| If the manual setup is too much hassle for you, you can use a docker based dev environment instead by following the instructions: | ||
|
|
||
| - Clone the repo and navigate to the root of the repo. | ||
| - Run `cp /docker/.env.sample /docker/.env` |
There was a problem hiding this comment.
I don't think there's a need for an env file during development to be honest. Hoarder should work just fine without one.
There was a problem hiding this comment.
This ties back into the above comment: "Wouldn't it be a good idea to set it up anyways, just to make sure that any development does not break functionality offered by nextauth or meilisearch?"
There was a problem hiding this comment.
coming back to this: wouldnt we need it for NEXTAUTH_URL and MEILISEARCH_URL?
|
@MohamedBassem Let me know if theres anything I can do to push this forward! |
|
The env file interferes with several homelab server distros |
This is tackling issue #580, to update the docker compose development documentation to make it more reliable for people to use and set up.