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

Firestore: Re-write API docs for COUNT API #6632

Merged
merged 6 commits into from
Sep 27, 2022

Conversation

dconeybe
Copy link
Contributor

The wording is copied from firebase/firebase-android-sdk#4143

@dconeybe dconeybe self-assigned this Sep 27, 2022
@changeset-bot
Copy link

changeset-bot bot commented Sep 27, 2022

⚠️ No Changeset found

Latest commit: b5af575

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@dconeybe dconeybe changed the base branch from master to mila/count-export-count-quries September 27, 2022 16:06
@google-oss-bot
Copy link
Contributor

google-oss-bot commented Sep 27, 2022

Size Report 1

Affected Products

  • @firebase/firestore

    TypeBase (2d55fc6)Merge (2235c77)Diff
    browser265 kB265 kB-40 B (-0.0%)
    esm5329 kB329 kB-44 B (-0.0%)
    main530 kB530 kB-99 B (-0.0%)
    module265 kB265 kB-40 B (-0.0%)
    react-native265 kB265 kB-40 B (-0.0%)
  • @firebase/firestore-lite

    TypeBase (2d55fc6)Merge (2235c77)Diff
    browser82.1 kB82.1 kB-40 B (-0.0%)
    esm598.3 kB98.3 kB-44 B (-0.0%)
    main139 kB139 kB-99 B (-0.1%)
    module82.1 kB82.1 kB-40 B (-0.0%)
    react-native82.3 kB82.3 kB-40 B (-0.0%)
  • firebase

    TypeBase (2d55fc6)Merge (2235c77)Diff
    firebase-firestore-lite.js87.8 kB87.7 kB-40 B (-0.0%)
    firebase-firestore.js313 kB313 kB-40 B (-0.0%)

Test Logs

  1. https://storage.googleapis.com/firebase-sdk-metric-reports/0TSfPZd23a.html

@google-oss-bot
Copy link
Contributor

google-oss-bot commented Sep 27, 2022

Size Analysis Report 1

This report is too large (211,141 characters) to be displayed here in a GitHub comment. Please use the below link to see the full report on Google Cloud Storage.

Test Logs

  1. https://storage.googleapis.com/firebase-sdk-metric-reports/2G7CxiXqBR.html

@github-actions github-actions bot added the doc-changes PRs that affect docs label Sep 27, 2022
@dconeybe dconeybe marked this pull request as ready for review September 27, 2022 18:35
@dconeybe dconeybe requested review from milaGGL and removed request for dwyfrequency and hsubox76 September 27, 2022 18:35
Copy link
Contributor

@milaGGL milaGGL left a comment

Choose a reason for hiding this comment

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

LGTM

@milaGGL milaGGL merged commit 88d0266 into mila/count-export-count-quries Sep 27, 2022
@milaGGL milaGGL deleted the dconeybe/CountApiDocs branch September 27, 2022 21:13
@firebase firebase locked and limited conversation to collaborators Oct 28, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
api: firestore doc-changes PRs that affect docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants