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

Grading PR #26

Open

opes wants to merge 1 commit into Vibn-App:main from opes:grading

opes commented Oct 21, 2021

Vibn Backend Grading Notes

Note: Using the split view in the "Files Changed" tab of this PR will make it easier to see grading comments & suggestions.

DO NOT MERGE THIS PR

Notes

Excellent work!
Make sure you fill out your .env-example
Tests are pretty minimal and need to be built out further
Make sure you're using the service layer to help organize your code

Overall, really solid work! Nice job!!


          Grading PR

a2a1913

opes marked this pull request as ready for review

October 21, 2021 19:56

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

Labels

None yet