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

Bug 1357476 - Replace django-rest-swagger with native d-r-f docs #3230

Merged
merged 1 commit into from Feb 16, 2018

Conversation

edmorley
Copy link
Contributor

Now that django-rest-framework supports interactive docs natively, we no longer need to use the additional django-rest-swagger package:
http://www.django-rest-framework.org/topics/3.6-announcement/#interactive-api-documentation
http://www.django-rest-framework.org/topics/documenting-your-api/

This significantly reduces the number of static files that have to be processed, reducing the time taken for collectstatic (which is run on every deployment and takes a while due to Brotli compression) by 70%.

Now that django-rest-framework supports interactive docs natively,
we no longer need to use the additional django-rest-swagger package:
http://www.django-rest-framework.org/topics/3.6-announcement/#interactive-api-documentation
http://www.django-rest-framework.org/topics/documenting-your-api/

This significantly reduces the number of static files that have to
be processed, reducing the time taken for `collectstatic` (which is
run on every deployment and takes a while due to Brotli compression)
by 70%.
@edmorley edmorley self-assigned this Feb 15, 2018
@edmorley edmorley merged commit 0aec43a into master Feb 16, 2018
@edmorley edmorley deleted the replace-swagger branch February 16, 2018 10:21
edmorley added a commit that referenced this pull request Apr 3, 2018
edmorley added a commit that referenced this pull request Apr 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant