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

Add infoToFindParams utils #86

Open
edobrb opened this issue Jan 26, 2022 · 2 comments
Open

Add infoToFindParams utils #86

edobrb opened this issue Jan 26, 2022 · 2 comments
Labels
💫 improvements Improvements or addition to existing functionalities

Comments

@edobrb
Copy link
Collaborator

edobrb commented Jan 26, 2022

It should converts the GraphQLResolverInfo into a FindParams accepted by the findAll operation
@edobrb edobrb added the 💫 improvements Improvements or addition to existing functionalities label Jan 26, 2022
@minox86
Copy link
Collaborator

minox86 commented Feb 1, 2022

I think this could be higher priority to avoid a lot of breaking changes (or unnecessary code) for the first users.

@minox86 minox86 added this to the Late February 2022 milestone Feb 1, 2022
@minox86
Copy link
Collaborator

minox86 commented Feb 1, 2022

We could evaluate the replacement of the infoToProjections function with the following librayr https://www.npmjs.com/package/graphql-fields to decrease technical debt.

@minox86 minox86 removed this from the September 2022 milestone May 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
💫 improvements Improvements or addition to existing functionalities
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@minox86 @edobrb