Skip to content

Conversation

@maneesht
Copy link
Collaborator

@maneesht maneesht commented Apr 7, 2025

No description provided.

@docs-page
Copy link

docs-page bot commented Apr 7, 2025

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

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

Documentation is deployed and generated using docs.page.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR introduces comprehensive documentation for using TanStack Query Firebase with Angular, complementing the existing React documentation. Key changes include:

  • Fixing a minor spelling error in the React querying docs.
  • Adding new Angular documentation pages for setup, querying, mutations, and function references.
  • Updating the global docs navigation to include Angular-specific content.

Reviewed Changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
docs/react/data-connect/querying.mdx Fixed a spelling mistake in the introductory paragraph.
docs/angular/index.mdx Added a new Angular landing page with setup instructions.
docs/angular/data-connect/querying.mdx Introduced Angular querying documentation with code examples.
docs/angular/data-connect/mutations.mdx Added mutations documentation with code examples (note potential class declaration syntax issue).
docs/angular/data-connect/index.mdx Provided the main guide for Firebase Data Connect using Angular.
docs/angular/data-connect/functions/injectDataConnectQuery.mdx Documented the usage of the injectDataConnectQuery function.
docs/angular/data-connect/functions/injectDataConnectMutation.mdx Documented the usage of the injectDataConnectMutation function (note potential class declaration syntax issue).
docs.json Updated the sidebar navigation to include Angular and its related docs.

Copy link
Collaborator

@cabljac cabljac left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@github-project-automation github-project-automation bot moved this to Approved [PR] in [Cloud] Open Source P0 May 8, 2025
@cabljac cabljac merged commit 53d0756 into invertase:main May 8, 2025
2 checks passed
@github-project-automation github-project-automation bot moved this from Approved [PR] to Done in [Cloud] Open Source P0 May 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants