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

Update view.py #809

Open
wants to merge 64 commits into
base: master
Choose a base branch
from
Open

Update view.py #809

wants to merge 64 commits into from

Conversation

lopezvit
Copy link

This relays on #730 and #499 but with the newest code, as I need to implement it in my own project.

WasinTh and others added 30 commits August 25, 2019 13:36
`permission_classes` expects tuple but list was provided
fix a bug: url from swagger_settings.DEFAULT_API_URL is not working
Check for allow_null attribute
This change fixes an `OPTIONS` request to the API page which will return
a successful response, but the `TemplateHTMLRender` will not be prepared
to render it since there is no template name provided in either the view
or response [1].

[1]: https://github.com/encode/django-rest-framework/blob/f0a5b958a134e8cd94e3ef3263e8fa623ac9b82f/rest_framework/renderers.py#L178-L189
Add utf-8 support for generated formats
Remove universal wheel, python 2 is unsupported
JoelLefkowitz and others added 25 commits July 17, 2022 20:19
Added `many` support to example code.
…integration

Drf extra fields base64 integration
Update permission_classes to a tuple
Add prepublish compatibility checks, a composite installation action and explicitly set tox targets
…ering-with-TemplateHTMLRenderer

handle errors rendering with TemplateHTMLRenderer
…one-field

add support for working with pytz object field
Add checker for allow_null attr
This commit will make sure that `Django :: 3.2` will show up in the
classifiers list on PyPI: https://pypi.org/project/drf-yasg/

(The magic happens because we parse the README to get supported Django):
https://github.com/DavidCain/drf-yasg/blob/ee29412d3cdb311/setup.py#L36

3.2 support should already exist
================================
A closed pull request, axnsan12#735,
noted support for Django 3.2 in both `tox.ini`, and the README.

That PR was closed in favor of
axnsan12#741, which edited `tox.ini`
and switched to GitHub Actions.

axnsan12#735 (comment)

This project has been testing on Django 3.2 for a long time (about a
year). I think we can declare it supported it the README!
On `zsh` (a popular shell, and the default for macOS), install fails:

    $ pip install -U -e .[validation]
    zsh: no matches found: .[validation]

Quote the arguments so that `bash` or `zsh` will succeed.
Declare Django 3.2 support in README, classifiers
Fix `pip install` command for contributors on zsh
This relays on axnsan12#730 and axnsan12#499 but with the newest code, as I need to implement it in my own project.
@JoelLefkowitz JoelLefkowitz deleted the branch axnsan12:master October 17, 2024 11:55
@JoelLefkowitz JoelLefkowitz reopened this Oct 17, 2024
@JoelLefkowitz JoelLefkowitz changed the base branch from 1.21.x to master October 17, 2024 12:02
@JoelLefkowitz JoelLefkowitz added the 1.23.x Release target in 1.23.x label Oct 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1.23.x Release target in 1.23.x
Projects
None yet
Development

Successfully merging this pull request may close these issues.