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

fix: Warden no longer available in latest graphql gem versions #278

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

camero2734
Copy link

@camero2734 camero2734 commented Aug 12, 2024

Fixes #276

As explained in the issue, in the latest versions of graphql, the warden object is no longer available in most places.

It's mostly replaced by @types -- which I initially used in the first pass -- but since the warden object can still be constructed, doing so was the easiest route for backwards compatibility purposes.

@camero2734 camero2734 marked this pull request as ready for review August 12, 2024 17:18
jeantessier added a commit to jeantessier/book-reviews that referenced this pull request Aug 15, 2024
    for service in books reviews users search signatures jwts
    do
        echo '==========' $service '=========='
        (cd $service; bundle update --all)
    done
    echo '========== gateway =========='
    (cd gateway; npm update)

!!!  BROKEN  !!!  BROKEN  !!!  BROKEN  !!!  BROKEN  !!!  BROKEN  !!!

It updated the `graphql` gem, but it still uses the same version of
the `apollo-federation` gem.  I might have to wait for that one to
get an upgrade before things start working again.

See Gusto/apollo-federation-ruby#276
and Gusto/apollo-federation-ruby#278
@comatory
Copy link

🙏 We're running into issues with graphql not being compatible. Would really appreciate fixing this. 🙇

@arinhouck
Copy link

This is blocking our upgrade of graphql-ruby as well. 🙇 Would be very grateful for a resolution on this one soon, anything else needed to get this moving forward?

@vaclavbohac
Copy link

vaclavbohac commented Aug 29, 2024

I've made an experiment where I forked this repo and made it run with the latest graphql-ruby package. I discovered there are many other problems par this one. If we are able to fix them, we can open a PR upstream.

productboardlabs#1

Here's an example of the test run https://app.circleci.com/pipelines/github/productboardlabs/apollo-federation-ruby/4/workflows/b750a570-c50b-41f6-b658-10c91871e1bc/jobs/88

@Bajena
Copy link

Bajena commented Sep 10, 2024

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.

graphql 2.3.8 removes "warden", breaking schema composition etc
5 participants