search

invinst / invisible-flow

9 stars 2 forks source link

[#155] add DB upgrade command to Dockerfile and manage.py; update REA… #128

Closed adam-kaplan-thoughtworks closed 4 years ago

adam-kaplan-thoughtworks commented 4 years ago

…DME to clarify purpose of SQL file on Dropbox

adam-kaplan-thoughtworks commented 4 years ago

repro steps:

  1. clone this project's repository into a new directory on your computer
  2. cd into that directory, git checkout ak/155 so you're using this branch's code, and go through the onboarding instructions in the readme up to "check if the app is running correctly"
  3. "check if the app is running correctly" instructions might warrant some rewriting, but once you get to this section, you will want to visit localhost:5000, run the scraper, and watch the database fill with things by performing the commands suggested in the current version of the readme.

expected: database has records

I was able to get thru testing but had to backtrack and do things in the readme that i didn't do the first time. I also got tripped up by running Docker from the wrong directory, but that is likely an Adam specific problem.

reblg commented 4 years ago

@adam-kaplan-thoughtworks can you actually set up some time with me to go over this so i know exactly what needs to be changed in the readme based on these changes.

adam-kaplan-thoughtworks commented 4 years ago

All good after a short convo