Fix for import from graphql-relay-py (#329) by Cito · Pull Request #330 · graphql-python/graphene-sqlalchemy · GitHub
Skip to content

Fix for import from graphql-relay-py (#329)#330

Merged
erikwrede merged 2 commits into
graphql-python:masterfrom
Cito:fix-for-graphql-relay
Apr 27, 2022
Merged

Fix for import from graphql-relay-py (#329)#330
erikwrede merged 2 commits into
graphql-python:masterfrom
Cito:fix-for-graphql-relay

Conversation

@Cito

@Cito Cito commented Jan 28, 2022

Copy link
Copy Markdown
Member

No description provided.

@codecov-commenter

codecov-commenter commented Jan 28, 2022

Copy link
Copy Markdown

@conao3

conao3 commented Mar 9, 2022

Copy link
Copy Markdown

Cito added 2 commits April 27, 2022 21:28
The module name was deprecated, but all imports
should be made from the top level anyway.
@erikwrede erikwrede force-pushed the fix-for-graphql-relay branch from 4392985 to 6fe8e6a Compare April 27, 2022 19:28
@erikwrede erikwrede merged commit 771f4f5 into graphql-python:master Apr 27, 2022
erikwrede added a commit to erikwrede/graphene-sqlalchemy that referenced this pull request Jun 9, 2022
….key instead of Column.name, see graphql-python#330

Signed-off-by: Erik Wrede <erikwrede2@gmail.com>
erikwrede added a commit that referenced this pull request Jul 15, 2022
* Fields generated from Hybrid Properties & Type hints now support Unions (Union[ObjectType1,OT2] or ObjectType1 | OT2)
* Support for Variant and types.JSON Columns
* BREAKING: Date&Time now convert to their corresponding graphene scalars instead of String.
* BREAKING: PG UUID & sqlalchemy_utils.UUIDType now convert to graphene.UUID instead of graphene.String
* Change: Sort Enums & ChoiceType enums are now generated from Column.key instead of Column.name, see #330

Signed-off-by: Erik Wrede <erikwrede2@gmail.com>
Co-authored-by: Nicolas Delaby <nicolas.delaby@infarm.com>
Co-authored-by: davidcim <david@cimaware.com>
Co-authored-by: Viktor Pegy <xtpvxt@gmail.com>
Co-authored-by: Ian Epperson <ian@epperson.com>
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.

4 participants