Skip to content

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

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 2 commits into from
Apr 27, 2022

Conversation

Cito
Copy link
Member

@Cito Cito commented Jan 28, 2022

No description provided.

@codecov-commenter
Copy link

codecov-commenter commented Jan 28, 2022

Codecov Report

Merging #330 (6fe8e6a) into master (7bf0aa5) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #330   +/-   ##
=======================================
  Coverage   96.91%   96.91%           
=======================================
  Files           9        9           
  Lines         616      616           
=======================================
  Hits          597      597           
  Misses         19       19           
Impacted Files Coverage Δ
graphene_sqlalchemy/fields.py 99.02% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7bf0aa5...6fe8e6a. Read the comment docs.

@conao3
Copy link

conao3 commented Mar 9, 2022

I want to this patch for my production source code to reduce unneeded warning. Any blockers?

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>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants