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

Set prefix permission #71

Merged
merged 6 commits into from
Feb 3, 2022
Merged

Set prefix permission #71

merged 6 commits into from
Feb 3, 2022

Conversation

chrisarmstrong151
Copy link
Collaborator

Re-factor some of the views under prefixes. The logic is cleaner now. The Swagger schema documentation could still use some work.

@HadleyKing HadleyKing self-requested a review January 26, 2022 13:19
@HadleyKing HadleyKing self-assigned this Jan 26, 2022
@HadleyKing HadleyKing added this to the 22.01 milestone Jan 26, 2022
@chrisarmstrong151
Copy link
Collaborator Author

Closing so that I can merge the migrations in.

@chrisarmstrong151
Copy link
Collaborator Author

Re-opening because the migrations folder looks correct now.

@HadleyKing HadleyKing self-requested a review January 29, 2022 13:48
@HadleyKing HadleyKing modified the milestones: 22.01, 22.02 Jan 31, 2022
- Fix 0001_initial.py to include original date
- linting for POST_api_prefixes_create.py
- linting for POST_api_prefixes_delete.py
- fix swagger for POST_api_prefixes_delete in views.py
Changes to be committed:
	modified:   bco_api/api/scripts/method_specific/POST_api_prefixes_modify.py
	modified:   bco_api/api/views.py
Changes to be committed:
	modified:   bco_api/api/scripts/method_specific/POST_api_prefixes_create.py
	modified:   bco_api/api/scripts/method_specific/POST_api_prefixes_token.py
	modified:   bco_api/api/scripts/method_specific/POST_api_prefixes_token_flat.py
	modified:   bco_api/api/views.py
@HadleyKing HadleyKing merged commit fdaacf1 into main Feb 3, 2022
@HadleyKing HadleyKing deleted the set_prefix_permission branch February 3, 2022 22:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants