Migrate to use upstream GraphQL

[wtrocki]

Currently we are using Chirino fork of the GraphQL.
GraphQL implementation moved forward we should investigate how much work will be needed to migrate

[ankithans]

Can I start working on it?
There are some extra files and folders in Chirino's fork. If we switch to upstream then there will be issues ig

[ankithans]

The version which we are using has some additional go modules but the upstream version doesn't have it. And those modules are used widely in the project. So what should I do for them now?

I can list the modules that are not in upstream if required.

Chirino version was forked at this point and then he added modules over it https://github.com/graph-gophers/graphql-go/tree/25d6d94fa7a7f2b0ce10fd509e54e85f7a2f866b

our	upstream

[wtrocki]

Ok. I'm aware of this, but what you expect me to do with this info? What is your question?
I would start with fixing CORS or JSON parse issues to get more familiar with the project.

[ankithans]

So my question is when I replace the imports everywhere from github.com/chirino/graphql to github.com/graph-gophers/graphql-go, there will be errors coming up for the modules which are not there.

So how should I include those modules in our project? Like should I add them as modules in graphql-link ?

[wtrocki]

This is low level implementation question. I guess you need those from somewhere.

[ankithans]

Yeah, they are available in github.com/chirino/graphql. I can include and update the necessary ones in graphql-link. If that sound fine?

[wtrocki]

Not sure what you mean TBH. Let's create PR with explanation what was done.

[ankithans]

Okay sure @wtrocki, will create PR soon. After working on the issue, I will also have more clarity.

[wtrocki]

Trust me - it will not be soon. There is more than week of work on migration IMHO.
Please try some easiest issues first

[ankithans]

yeah realizing 🥲

Please try some easiest issues first

okay, got it!

[namit-chandwani]

@wtrocki Can you please assign this issue to me? Will start working on it right away