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

[WIP] DRF uplift #585

Closed
wants to merge 13 commits into from
Closed

[WIP] DRF uplift #585

wants to merge 13 commits into from

Conversation

Pomax
Copy link
Contributor

@Pomax Pomax commented Dec 16, 2019

Closes #544

Clean uplift to 3.6.3, "dirty" uplift for 3.6.4->3.10.3 due to the change to the filter code that landed in encode/django-rest-framework#5264

PR on the DRF-side to hopefully look into this: encode/django-rest-framework#7094

@@ -4,26 +4,26 @@ verify_ssl = true
name = "pypi"

[packages]
"psycopg2-binary" = "*"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

VS Code's "sort alphabetically"

@@ -2,6 +2,7 @@


class NewsEntrySerializer(EntrySerializerWithCreators):
help_types = None
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

required by 3.9

@@ -248,7 +261,7 @@ def get_serializer_context(self):
# When people POST to this route, we want to do some
# custom validation involving CSRF and nonce validation,
# so we intercept the POST handling a little.
@detail_route(methods=['post'])
@action(detail=True, methods=['post'])
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

required by 3.9

@@ -428,7 +441,7 @@ def get_serializer_context(self):
# When people POST to this route, we want to do some
# custom validation involving CSRF and nonce validation,
# so we intercept the POST handling a little.
@detail_route(methods=['post'])
@action(detail=True, methods=['post'])
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

required by 3.9

@Pomax Pomax added this to the Jan 20 milestone Dec 19, 2019
@alanmoo alanmoo removed this from the Jan 20 milestone Jan 7, 2020
@cadecairos cadecairos added this to the Icebox milestone Feb 25, 2020
@Pomax
Copy link
Contributor Author

Pomax commented Feb 26, 2020

superceded by #618

@Pomax Pomax closed this Feb 26, 2020
@Pomax Pomax deleted the drf-uplift-again branch February 26, 2020 20:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

uplift DRF to the latest version (epic)
3 participants