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.

Sign up for GitHub

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

Jump to bottom

GQLFRParams #64

Closed
bsr203 opened this issue Nov 6, 2015 · 3 comments
Closed

GQLFRParams #64

bsr203 opened this issue Nov 6, 2015 · 3 comments
Labels
enhancement

Comments

@bsr203
Copy link

bsr203 commented Nov 6, 2015

After the major rewrite, please consider

GQLFRParams -> ResolveParams / Params

there is also a TODO in there

TODO: clean up GQLFRParams fields

please consider what to be done, if none, may remove it.
@bsr203 bsr203 mentioned this issue Nov 6, 2015
Open
@pkieltyka
Copy link

That would be nice, I agree its already in the graphql package.

@sogko
Copy link
Member

sogko commented Nov 10, 2015

Updates: Addressed in #68

@sogko
Copy link
Member

sogko commented Nov 18, 2015

Resolved in #68.

Thanks guys!
Cheers!

@sogko sogko closed this as completed Nov 18, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@pkieltyka @sogko @bsr203