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

Ft test get persons #32

Open

cdvx wants to merge 1 commit into develop from ft-test-get-persons

Contributor

cdvx commented Jul 30, 2019 •

edited

Loading

What does this PR do?

Adds the roles app

Description of Task to be completed?

Enable users to view users/user
This was already implemented but untested
Update the persons model and change name attribute to nickname since first and last name attributes already exist

How should this be manually tested?

Fetch the branch with git fetch origin ft-test-get-persons
Run migrations and upgrade db python manage.py db migrate && python manage.py db upgrade
Run the app
Test the modifications to the endpoints for getting resources

Hit /persons , or /persons/<person_id endpoints to test the functionality
Run tests

closes #22


Test get persons

e1640f1

cdvx force-pushed the ft-test-get-persons branch from 0fe6916 to e1640f1 Compare

July 30, 2019 11:47

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