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

Django v4, python 3.10 support for graphene-django v3 #1281

Merged

Conversation

tim-schilling
Copy link
Contributor

This squashes and ports #1275, then fixes a few small issues for django main and python 3.10

MisterGlass and others added 3 commits December 8, 2021 18:47
Replace calls to deprecated methods

Fix test config & Replace additional methods removed in django 4.0

Update tox for official Django 4 release
This is being removed in django 4.1
@tim-schilling tim-schilling mentioned this pull request Dec 9, 2021
7 tasks
@tim-schilling
Copy link
Contributor Author

tim-schilling commented Dec 9, 2021 via email

ulgens
ulgens previously requested changes Dec 11, 2021
tox.ini Show resolved Hide resolved
tox.ini Outdated Show resolved Hide resolved
graphene_django/tests/urls_pretty.py Outdated Show resolved Hide resolved
@devkral
Copy link

devkral commented Dec 11, 2021

Ah, good catch.

On Thu, Dec 9, 2021, 3:23 PM Alexander @.***> wrote: the force_text to force_str patches are missing — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#1281 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAJYZP24HP7SBOW35VGKBVTUQEM4FANCNFSM5JVE6FLQ .

sry, it was a false alarm. This is why I removed my comment. The str_text patch is already included in master

The DEFAULT_GRAPHQL_URL had to change because it doesn't take into
account the APPEND_SLASH setting from within Django.
Copy link
Collaborator

@keithhackbarth keithhackbarth left a comment

Choose a reason for hiding this comment

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

This comment looks solid to me. All of @ulgens feedback appears to be addressed. I don't see any additional issues. Great work @tim-schilling

@keithhackbarth keithhackbarth requested review from ulgens and removed request for ulgens January 6, 2022 22:57
Copy link
Member

@jkimbo jkimbo left a comment

Choose a reason for hiding this comment

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

Looks great 👍

@jkimbo jkimbo merged commit 5d5d7f1 into graphql-python:main Jan 7, 2022
@djstein
Copy link

djstein commented Jun 23, 2022

@jkimbo can we get a beta 3.0.0b8 of this? this has the fixes I require for Python 3.10 usage

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants