search

meshcenter / mesh-api

Database for managing a mesh network
https://api.nycmesh.net
GNU Affero General Public License v3.0
17 stars 10 forks source link

Add initial database schema migrations #70

Closed greenberga closed 3 years ago

greenberga commented 3 years ago

This contains a port of scripts/create-database.js to node-pg-migrate. I also added the start of a docker-compose.yml to simplify local testing.

Before merging, let’s test this in staging!

Closes #64

greenberga commented 3 years ago

If desired, I can add a commit that deletes scripts/create-database.js too 🙂

alexbbt commented 3 years ago

Deleting that file makes sense, but let's wait until everyone gets moved over to this new system

alexbbt commented 3 years ago

I made a new issue to handle deleting that file and connecting to the new Database #71

alexbbt commented 3 years ago

After further review, your PR seems to fully fulfill this, go ahead and delete that file and I'll close that issue

greenberga commented 3 years ago

I would prefer if you updated the white space on the schema file to be normal indentation, otherwise it looks great!

Going to follow up with a PR that formats all the JS files using Prettier, and adds it as a devDependency + a script in package.json.