Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat(fdc)!: breaking change - Introduce dataConnectResult field to provide fields that were overridden when spreading data #172

Merged
merged 72 commits into from
Mar 18, 2025

Conversation

maneesht
Copy link
Contributor

@maneesht maneesht commented Mar 13, 2025

If a user has a field that conflicts with QueryResult or MutationResult fields such as ref, source, they get overridden. This PR includes a new dataConnectResult field that allows us to expose the entire QueryResult/MutationResult to users. Additionally, we removed all sibling fields of data, to reduce other conflicts.

const { dataConnectResult } = useListMovies();

dataConnectResult.ref // reference to the `ListMovies` operation's ref

Copy link

docs-page bot commented Mar 13, 2025

To view this pull requests documentation preview, visit the following URL:

docs.page/invertase/tanstack-query-firebase~172

Documentation is deployed and generated using docs.page.

@maneesht maneesht changed the title feat(fdc): Introduce resultMeta field to provide fields that were overridden when spreading data feat(fdc): breaking change - Introduce dataConnectResult field to provide fields that were overridden when spreading data Mar 14, 2025
@yuchenshi
Copy link

This is good to go. I saw you've updated the title but don't forget to update the PR description to match the updated API too @maneesht

@maneesht
Copy link
Contributor Author

Done @yuchenshi. Thanks!

@Ehesp Ehesp changed the title feat(fdc): breaking change - Introduce dataConnectResult field to provide fields that were overridden when spreading data feat(fdc)!: breaking change - Introduce dataConnectResult field to provide fields that were overridden when spreading data Mar 18, 2025
@Ehesp Ehesp merged commit 0feb40a into invertase:main Mar 18, 2025
3 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants