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.

Sign up for GitHub

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

Jump to bottom

Code Review #2

Open
droxey opened this issue Dec 17, 2019 · 0 comments
Open

Code Review #2

droxey opened this issue Dec 17, 2019 · 0 comments

Comments

@droxey
Copy link

droxey commented Dec 17, 2019

FEEDBACK:

  • Add env/ to .gitignore --- never commit your virtual environment!
  • Great work!
Criteria Possible ✓
At least 2 apps in project 40 40
At least 2 tests in tests.py in each app 30 30
Modular design 25 25
RESTful API using Django REST Framework 25 25
Template-based front-end interface 25 25
Has purpose & fulfills proposal 20 20
Integrates an open source app 20 20
Deployed & usable 15 15
Includes database migrations in each app 15 15
README with documentation 10 10
Public GitHub repo 5 5
TOTAL 230 230
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@droxey