Skip to content
This repository has been archived by the owner on Jan 30, 2020. It is now read-only.

Fix method name clash #65

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

maxhungry
Copy link

graphql-ruby v1.8.7 introduced a private method with the same name
apply_scope in GraphQL::Schema::Field, which happens to be where
this module was designed to be prepended to.

Close #63

graphql-ruby v1.8.7 introduced a private method with the same name
`apply_scope` in GraphQL::Schema::Field, which happens to be where
this module was designed to be prepended to.

Close ontohub#63
@schonert
Copy link

@phyrog any chance of you having time to go through this anytime soon?

@phyrog
Copy link
Collaborator

phyrog commented Dec 10, 2018

@schonert thanks for the ping. As I am no longer actively involved with Ontohub I did not get a notification about this. I will have a look at it tomorrow

@schonert
Copy link

@phyrog let me know if i can be of any help

@v-kolesnikov
Copy link

v-kolesnikov commented Jan 4, 2019

Hi folks! Any news with it? I tried to use this library and came across to this issue too( I also paid attention on that tests can't be passed on the master branch. How can I help you with it? Feel free to ask for help if it's needed. I'm continuing explore the sources anyway. cc @phyrog

@durcak
Copy link

durcak commented Jan 28, 2019

Any news with it? cc @phyrog @eugenk

Batou99 added a commit to conferize/graphql-pundit that referenced this pull request Mar 22, 2019
@durcak
Copy link

durcak commented Mar 28, 2019

Its fixed with new version of graphql gem 👍

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

Successfully merging this pull request may close these issues.

5 participants