Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update models #70

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Update models #70

wants to merge 9 commits into from

Conversation

gnocchi2815
Copy link
Member

@gnocchi2815 gnocchi2815 commented Mar 27, 2017

ISSUE NUMBER

#53 #62 #63 #64 #68

Labels

  • Add IN REVIEW to labels

Describe the changes

This pull request contains separate commits to address the issues above relating to the Cafe model:

  • The cafe ratings fields have been renamed.
  • The created_at field has been removed
  • The address model has been deleted and the address fields moved to the cafe model
  • The Atmosphere, Coffee and WiFi models have been removed and the foreign key relationship updated
  • timezone is no longer imported as it is not required for any of the remaining fields
  • Added a deleted boolean field to the Cafe model to enable soft delete of cafes
  • Migrations have been made for each of the changes.

Screenshots (if appropriate)

n/a

Questions or comments (if any)

n/a

@gnocchi2815
Copy link
Member Author

I have added in some filters based on the ratings, and associated tests for the filters

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant