gapi-to-graphql icon indicating copy to clipboard operation
gapi-to-graphql copied to clipboard

someEndpointsHaveNoGraphQLRepresentationAndMeDontLikeThat - Support for maps in the endpoints

Open adamisntdead opened this issue 7 years ago • 3 comments

Hey!

Just playing around with this today, and I was really just looking at the schemas being generated until I came across this gem:

type ruleGroups {
    thisTypeHasNoFieldsAndGraphQLDontLikeThat: Boolean
}

I'm going to have to be honest and say I don't really think that's gonna work, funny as it may be:

Can I make two suggestions on how to deal with this?

First, those types usually are just objects with keys, something like this in JSON:

(key): {
  ...
}

My suggestion would be turn that into a type like:

type {
  key: 'blahblah'
  ...
}

If that wouldn't work, I would change the initial type to something like:

type ruleGroups {
    thisTypeHasNoFieldsAndGraphQLDontLikeThat: Boolean
    json: String
}

And just do a JSON.stringify(...) and dump it in that json field

Thanks!

adamisntdead avatar Aug 03 '18 12:08 adamisntdead

@adamisntdead thanks for the feedback! I'm currently refactoring the code base to use TypeScript and will see if I come up with something more elegant for thisTypeHasNoFieldsAndGraphQLDontLikeThat, I initially added that in out of frustration but was always the goal to replace it with something more elegant.

rlancer avatar Aug 03 '18 16:08 rlancer

@adamisntdead which GAPI are you using?

rlancer avatar Aug 03 '18 16:08 rlancer

I was trying to use pagespeed (I ended up writing my own bindings only for the fields I needed)

adamisntdead avatar Aug 03 '18 16:08 adamisntdead