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.

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

fix get_template_context #9212

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

fix get_template_context #9212

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
01ea842
fix get_template_context
satayyeb Jan 5, 2024
3e45dd0
bugfix
satayyeb Jan 5, 2024
1df67ec
fix doc to pass pre-commit test
satayyeb Jan 5, 2024
ff61a16
pre-commit again :|
satayyeb Jan 5, 2024
f2fff36
Merge branch 'master' into fix-get_template_context
satayyeb Jan 27, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
4 changes: 2 additions & 2 deletions docs/topics/browsable-api.md
View file
Open in desktop
Expand Up @@ -21,8 +21,8 @@ To quickly add authentication to the browesable api, add a routes named `"login"

```python
urlpatterns = [
# ...
url(r"^api-auth/", include("rest_framework.urls", namespace="rest_framework"))
...,
url(r"^api-auth/", include("rest_framework.urls", namespace="rest_framework")),
]
```

Expand Down
2 changes: 2 additions & 0 deletions rest_framework/renderers.py
View file
Open in desktop
Expand Up @@ -170,6 +170,8 @@ def resolve_template(self, template_names):

def get_template_context(self, data, renderer_context):
response = renderer_context['response']
if isinstance(data, list):
return {'details': data, 'status_code': response.status_code}
if response.exception:
data['status_code'] = response.status_code
return data
Expand Down