search

CCI-MOC / flocx-market

2 stars 9 forks source link

Application rewritten to flask with a system test working that requires running db #8

Closed fvukelic closed 5 years ago

fvukelic commented 5 years ago

Tox and travis are failing. Feedback is appreciated.

tzumainn commented 5 years ago

Hi! This is impressive - nice work! My first high-level comment is that we probably want to reorganize some of the files to more closely match what we find in other OpenStack projects:

a) All the database stuff should be in db/ and db/migrations/ b) All the API stuff should be in api/ c) The unit test directory structure should match - so tests/db/, tests/api/, etc.