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

Output warning the graphql inference process drops fields due to type mis-matches #2405

Closed
KyleAMathews opened this issue Oct 9, 2017 · 2 comments
Closed

Output warning the graphql inference process drops fields due to type mis-matches #2405

KyleAMathews opened this issue Oct 9, 2017 · 2 comments
Labels
help wanted Issue with a clear description that the community can help with.

Comments

@KyleAMathews
Copy link
Contributor

Users are surprised sometimes due to fields in their data being dropped due to mis-matched types. Instead of silently dropping data, we should output a warning about the data source (file(s), external API, etc) and any transformers involved and examples of the mis-matched data.

#2404 is an example of an issue coming from the lack of this warning and there's been more in the past.

/cc @danoc @jquense
@KyleAMathews KyleAMathews added DX help wanted Issue with a clear description that the community can help with. labels Oct 9, 2017
@cricrio cricrio mentioned this issue Feb 10, 2018
Closed
@m-allanson
Copy link
Contributor

Related: #3905

@pieh
Copy link
Contributor

pieh commented Apr 15, 2018

Closed via #3905

@pieh pieh closed this as completed Apr 15, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
help wanted Issue with a clear description that the community can help with.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@KyleAMathews @m-allanson @pieh