Remove hardcoded database url and add DATABASE_HOST variable

[krumware]

This allows the startup script to use a runtime variable for control over the database location. Previously, the start.sh script was hardcoded to a specific database url specified by the service name in a docker-compose network.

closes #19

[krumware]

@vanceism7 wondering if you might be able to informally peer-review / try this change

[vanceism7]

Hey @krumware, don't have my computer on me currently. I'll try to give this a test soon though if no one else has

[vanceism7]

@krumware - Did we want to test this PR exactly as is or with that modified Docker file you supplied in that issue we were discussing? The change seems pretty straightforward but wanted to confirm this before testing

[krumware]

@vanceism7 either-or, since it's on the compose side.
I have a gut feeling that the quotes may need to be removed from the .env var, but it's working as-PRed for me

[PeerRich]

has someone tested this? is it working?

[vanceism7]

Sorry, it took a while to get to testing this. This is working for me, with the caveat that I had to follow the instructions discussed in #11 (e.g: replace the Dockerfile, remove quotes from the .env file, change yarn start to yarn dev in scripts/start.sh)

Since this change has strictly to do with the database env variables and not any of the issues specifically coming from #11, I'll give this my approval (although it should be noted that I'm not associated with this project in any official capacity).

[krumware]

that was the goal, to keep it small and on topic.
I'm waiting to see if there's any movement on removing the postgres requirement from the build before I'm able to finish the other docker pieces and will include a good number of polishing changes in a separate PR for that

[kelvinalfaro]

This worked for me too.

[kelvinalfaro]

These changes worked for me