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.

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

Feature Request: Add a type's dataId automatically #124

Open
baconcheese113 opened this issue Jul 17, 2022 · 1 comment
Open

Feature Request: Add a type's dataId automatically #124

baconcheese113 opened this issue Jul 17, 2022 · 1 comment

Comments

@baconcheese113
Copy link
Contributor

As mentioned in #123 it could help users avoid cache bugs and copy/pasting id throughout all their queries if it was added automatically to fields that don't specify it, just like how __typename is. No error is reported when a user doesn't add id yet their cache will fail to update, which can be unexpected behavior

It's a habit to add it to all fields for people that have been working with graphql for a while, but it's not immediately obvious to new graphql users. It would shrink queries and make it more apparent what fields the widget is actually using

@RobertBrunhage
Copy link

I just hit a couple of these exceptions like CacheMissException. Took a while to figure out that a specific query was missing the id field.

Highly agree that this would be super nice to have!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants