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

Implement Partial Errors for FDC #8821

Merged
merged 19 commits into from
Mar 21, 2025
Merged

Implement Partial Errors for FDC #8821

merged 19 commits into from
Mar 21, 2025

Conversation

maneesht
Copy link
Contributor

No description provided.

Copy link

changeset-bot bot commented Feb 28, 2025

🦋 Changeset detected

Latest commit: c695523

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@firebase/data-connect Patch
firebase Patch

Not sure what this means? Click here to learn what changesets are.

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

@google-oss-bot
Copy link
Contributor

google-oss-bot commented Feb 28, 2025

Size Report 1

Affected Products

  • @firebase/data-connect

    TypeBase (5718838)Merge (441a66d)Diff
    browser21.1 kB21.4 kB+305 B (+1.4%)
    main23.1 kB23.6 kB+529 B (+2.3%)
    module21.1 kB21.4 kB+305 B (+1.4%)
  • firebase

    TypeBase (5718838)Merge (441a66d)Diff
    firebase-data-connect.js17.3 kB17.6 kB+284 B (+1.6%)

Test Logs

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

@google-oss-bot
Copy link
Contributor

google-oss-bot commented Feb 28, 2025

Size Analysis Report 1

Affected Products

  • @firebase/data-connect

    • CallerSdkTypeEnum

      Size

      TypeBase (5718838)Merge (441a66d)Diff
      size12.4 kB12.6 kB+181 B (+1.5%)
      size-with-ext-deps30.6 kB30.8 kB+183 B (+0.6%)

      Dependency

      TypeBase (5718838)Merge (441a66d)Diff
      classes

      AppCheckTokenProvider
      DataConnect
      DataConnectError
      FirebaseAuthProvider
      MutationManager
      QueryManager
      RESTTransport

      AppCheckTokenProvider
      DataConnect
      DataConnectError
      DataConnectOperationError
      FirebaseAuthProvider
      MutationManager
      QueryManager
      RESTTransport

      + DataConnectOperationError

    • DataConnect

      Size

      TypeBase (5718838)Merge (441a66d)Diff
      size12.4 kB12.6 kB+181 B (+1.5%)
      size-with-ext-deps30.6 kB30.8 kB+183 B (+0.6%)

      Dependency

      TypeBase (5718838)Merge (441a66d)Diff
      classes

      AppCheckTokenProvider
      DataConnect
      DataConnectError
      FirebaseAuthProvider
      MutationManager
      QueryManager
      RESTTransport

      AppCheckTokenProvider
      DataConnect
      DataConnectError
      DataConnectOperationError
      FirebaseAuthProvider
      MutationManager
      QueryManager
      RESTTransport

      + DataConnectOperationError

    • DataConnectError

      Size

      TypeBase (5718838)Merge (441a66d)Diff
      size?12.6 kB? (?)
      size-with-ext-deps?30.8 kB? (?)

      Dependency

      TypeBase (5718838)Merge (441a66d)Diff
      functions?

      15 dependencies

      addToken
      areTransportOptionsEqual
      compareDates
      dcFetch
      getGoogApiClientValue
      getMessage
      getRefSerializer
      logDebug
      logError
      parseOptions
      registerDataConnect
      setEncoder
      setIfNotExists
      setSDKVersion
      urlBuilder

      ?
      classes?

      AppCheckTokenProvider
      DataConnect
      DataConnectError
      DataConnectOperationError
      FirebaseAuthProvider
      MutationManager
      QueryManager
      RESTTransport

      ?
      variables?

      12 dependencies

      CallerSdkTypeEnum
      Code
      FIREBASE_DATA_CONNECT_EMULATOR_HOST_VAR
      QUERY_STR
      SDK_VERSION
      SOURCE_CACHE
      SOURCE_SERVER
      connectFetch
      encoderImpl
      logger
      name
      version

      ?
      enums??

      External Dependency

      ModuleBase (5718838)Merge (441a66d)Diff
      @firebase/app?

      SDK_VERSION
      _isFirebaseServerApp
      _registerComponent
      _removeServiceInstance
      registerVersion

      ?
      @firebase/component?

      Component

      ?
      @firebase/logger?

      Logger

      ?
      @firebase/util?

      FirebaseError

      ?
    • DataConnectOperationError

      Size

      TypeBase (5718838)Merge (441a66d)Diff
      size?12.6 kB? (?)
      size-with-ext-deps?30.8 kB? (?)

      Dependency

      TypeBase (5718838)Merge (441a66d)Diff
      functions?

      15 dependencies

      addToken
      areTransportOptionsEqual
      compareDates
      dcFetch
      getGoogApiClientValue
      getMessage
      getRefSerializer
      logDebug
      logError
      parseOptions
      registerDataConnect
      setEncoder
      setIfNotExists
      setSDKVersion
      urlBuilder

      ?
      classes?

      AppCheckTokenProvider
      DataConnect
      DataConnectError
      DataConnectOperationError
      FirebaseAuthProvider
      MutationManager
      QueryManager
      RESTTransport

      ?
      variables?

      12 dependencies

      CallerSdkTypeEnum
      Code
      FIREBASE_DATA_CONNECT_EMULATOR_HOST_VAR
      QUERY_STR
      SDK_VERSION
      SOURCE_CACHE
      SOURCE_SERVER
      connectFetch
      encoderImpl
      logger
      name
      version

      ?
      enums??

      External Dependency

      ModuleBase (5718838)Merge (441a66d)Diff
      @firebase/app?

      SDK_VERSION
      _isFirebaseServerApp
      _registerComponent
      _removeServiceInstance
      registerVersion

      ?
      @firebase/component?

      Component

      ?
      @firebase/logger?

      Logger

      ?
      @firebase/util?

      FirebaseError

      ?
    • MUTATION_STR

      Size

      TypeBase (5718838)Merge (441a66d)Diff
      size12.4 kB12.6 kB+181 B (+1.5%)
      size-with-ext-deps30.6 kB30.8 kB+183 B (+0.6%)

      Dependency

      TypeBase (5718838)Merge (441a66d)Diff
      classes

      AppCheckTokenProvider
      DataConnect
      DataConnectError
      FirebaseAuthProvider
      MutationManager
      QueryManager
      RESTTransport

      AppCheckTokenProvider
      DataConnect
      DataConnectError
      DataConnectOperationError
      FirebaseAuthProvider
      MutationManager
      QueryManager
      RESTTransport

      + DataConnectOperationError

    • MutationManager

      Size

      TypeBase (5718838)Merge (441a66d)Diff
      size12.4 kB12.6 kB+181 B (+1.5%)
      size-with-ext-deps30.6 kB30.8 kB+183 B (+0.6%)

      Dependency

      TypeBase (5718838)Merge (441a66d)Diff
      classes

      AppCheckTokenProvider
      DataConnect
      DataConnectError
      FirebaseAuthProvider
      MutationManager
      QueryManager
      RESTTransport

      AppCheckTokenProvider
      DataConnect
      DataConnectError
      DataConnectOperationError
      FirebaseAuthProvider
      MutationManager
      QueryManager
      RESTTransport

      + DataConnectOperationError

    • QUERY_STR

      Size

      TypeBase (5718838)Merge (441a66d)Diff
      size12.4 kB12.6 kB+181 B (+1.5%)
      size-with-ext-deps30.6 kB30.8 kB+183 B (+0.6%)

      Dependency

      TypeBase (5718838)Merge (441a66d)Diff
      classes

      AppCheckTokenProvider
      DataConnect
      DataConnectError
      FirebaseAuthProvider
      MutationManager
      QueryManager
      RESTTransport

      AppCheckTokenProvider
      DataConnect
      DataConnectError
      DataConnectOperationError
      FirebaseAuthProvider
      MutationManager
      QueryManager
      RESTTransport

      + DataConnectOperationError

    • SOURCE_CACHE

      Size

      TypeBase (5718838)Merge (441a66d)Diff
      size12.4 kB12.6 kB+181 B (+1.5%)
      size-with-ext-deps30.6 kB30.8 kB+183 B (+0.6%)

      Dependency

      TypeBase (5718838)Merge (441a66d)Diff
      classes

      AppCheckTokenProvider
      DataConnect
      DataConnectError
      FirebaseAuthProvider
      MutationManager
      QueryManager
      RESTTransport

      AppCheckTokenProvider
      DataConnect
      DataConnectError
      DataConnectOperationError
      FirebaseAuthProvider
      MutationManager
      QueryManager
      RESTTransport

      + DataConnectOperationError

    • SOURCE_SERVER

      Size

      TypeBase (5718838)Merge (441a66d)Diff
      size12.4 kB12.6 kB+181 B (+1.5%)
      size-with-ext-deps30.6 kB30.8 kB+183 B (+0.6%)

      Dependency

      TypeBase (5718838)Merge (441a66d)Diff
      classes

      AppCheckTokenProvider
      DataConnect
      DataConnectError
      FirebaseAuthProvider
      MutationManager
      QueryManager
      RESTTransport

      AppCheckTokenProvider
      DataConnect
      DataConnectError
      DataConnectOperationError
      FirebaseAuthProvider
      MutationManager
      QueryManager
      RESTTransport

      + DataConnectOperationError

    • areTransportOptionsEqual

      Size

      TypeBase (5718838)Merge (441a66d)Diff
      size12.4 kB12.6 kB+181 B (+1.5%)
      size-with-ext-deps30.6 kB30.8 kB+183 B (+0.6%)

      Dependency

      TypeBase (5718838)Merge (441a66d)Diff
      classes

      AppCheckTokenProvider
      DataConnect
      DataConnectError
      FirebaseAuthProvider
      MutationManager
      QueryManager
      RESTTransport

      AppCheckTokenProvider
      DataConnect
      DataConnectError
      DataConnectOperationError
      FirebaseAuthProvider
      MutationManager
      QueryManager
      RESTTransport

      + DataConnectOperationError

    • connectDataConnectEmulator

      Size

      TypeBase (5718838)Merge (441a66d)Diff
      size12.5 kB12.6 kB+181 B (+1.5%)
      size-with-ext-deps30.7 kB30.9 kB+183 B (+0.6%)

      Dependency

      TypeBase (5718838)Merge (441a66d)Diff
      classes

      AppCheckTokenProvider
      DataConnect
      DataConnectError
      FirebaseAuthProvider
      MutationManager
      QueryManager
      RESTTransport

      AppCheckTokenProvider
      DataConnect
      DataConnectError
      DataConnectOperationError
      FirebaseAuthProvider
      MutationManager
      QueryManager
      RESTTransport

      + DataConnectOperationError

    • executeMutation

      Size

      TypeBase (5718838)Merge (441a66d)Diff
      size12.5 kB12.6 kB+181 B (+1.5%)
      size-with-ext-deps30.7 kB30.9 kB+183 B (+0.6%)

      Dependency

      TypeBase (5718838)Merge (441a66d)Diff
      classes

      AppCheckTokenProvider
      DataConnect
      DataConnectError
      FirebaseAuthProvider
      MutationManager
      QueryManager
      RESTTransport

      AppCheckTokenProvider
      DataConnect
      DataConnectError
      DataConnectOperationError
      FirebaseAuthProvider
      MutationManager
      QueryManager
      RESTTransport

      + DataConnectOperationError

    • executeQuery

      Size

      TypeBase (5718838)Merge (441a66d)Diff
      size12.4 kB12.6 kB+181 B (+1.5%)
      size-with-ext-deps30.7 kB30.9 kB+183 B (+0.6%)

      Dependency

      TypeBase (5718838)Merge (441a66d)Diff
      classes

      AppCheckTokenProvider
      DataConnect
      DataConnectError
      FirebaseAuthProvider
      MutationManager
      QueryManager
      RESTTransport

      AppCheckTokenProvider
      DataConnect
      DataConnectError
      DataConnectOperationError
      FirebaseAuthProvider
      MutationManager
      QueryManager
      RESTTransport

      + DataConnectOperationError

    • getDataConnect

      Size

      TypeBase (5718838)Merge (441a66d)Diff
      size13.1 kB13.3 kB+181 B (+1.4%)
      size-with-ext-deps38.4 kB38.6 kB+183 B (+0.5%)

      Dependency

      TypeBase (5718838)Merge (441a66d)Diff
      classes

      AppCheckTokenProvider
      DataConnect
      DataConnectError
      FirebaseAuthProvider
      MutationManager
      QueryManager
      RESTTransport

      AppCheckTokenProvider
      DataConnect
      DataConnectError
      DataConnectOperationError
      FirebaseAuthProvider
      MutationManager
      QueryManager
      RESTTransport

      + DataConnectOperationError

    • mutationRef

      Size

      TypeBase (5718838)Merge (441a66d)Diff
      size12.5 kB12.7 kB+181 B (+1.4%)
      size-with-ext-deps30.7 kB30.9 kB+183 B (+0.6%)

      Dependency

      TypeBase (5718838)Merge (441a66d)Diff
      classes

      AppCheckTokenProvider
      DataConnect
      DataConnectError
      FirebaseAuthProvider
      MutationManager
      QueryManager
      RESTTransport

      AppCheckTokenProvider
      DataConnect
      DataConnectError
      DataConnectOperationError
      FirebaseAuthProvider
      MutationManager
      QueryManager
      RESTTransport

      + DataConnectOperationError

    • parseOptions

      Size

      TypeBase (5718838)Merge (441a66d)Diff
      size12.4 kB12.6 kB+181 B (+1.5%)
      size-with-ext-deps30.6 kB30.8 kB+183 B (+0.6%)

      Dependency

      TypeBase (5718838)Merge (441a66d)Diff
      classes

      AppCheckTokenProvider
      DataConnect
      DataConnectError
      FirebaseAuthProvider
      MutationManager
      QueryManager
      RESTTransport

      AppCheckTokenProvider
      DataConnect
      DataConnectError
      DataConnectOperationError
      FirebaseAuthProvider
      MutationManager
      QueryManager
      RESTTransport

      + DataConnectOperationError

    • queryRef

      Size

      TypeBase (5718838)Merge (441a66d)Diff
      size12.5 kB12.7 kB+181 B (+1.4%)
      size-with-ext-deps30.7 kB30.9 kB+183 B (+0.6%)

      Dependency

      TypeBase (5718838)Merge (441a66d)Diff
      classes

      AppCheckTokenProvider
      DataConnect
      DataConnectError
      FirebaseAuthProvider
      MutationManager
      QueryManager
      RESTTransport

      AppCheckTokenProvider
      DataConnect
      DataConnectError
      DataConnectOperationError
      FirebaseAuthProvider
      MutationManager
      QueryManager
      RESTTransport

      + DataConnectOperationError

    • setLogLevel

      Size

      TypeBase (5718838)Merge (441a66d)Diff
      size12.4 kB12.6 kB+181 B (+1.5%)
      size-with-ext-deps30.7 kB30.8 kB+183 B (+0.6%)

      Dependency

      TypeBase (5718838)Merge (441a66d)Diff
      classes

      AppCheckTokenProvider
      DataConnect
      DataConnectError
      FirebaseAuthProvider
      MutationManager
      QueryManager
      RESTTransport

      AppCheckTokenProvider
      DataConnect
      DataConnectError
      DataConnectOperationError
      FirebaseAuthProvider
      MutationManager
      QueryManager
      RESTTransport

      + DataConnectOperationError

    • subscribe

      Size

      TypeBase (5718838)Merge (441a66d)Diff
      size13.6 kB13.8 kB+181 B (+1.3%)
      size-with-ext-deps39.0 kB39.2 kB+183 B (+0.5%)

      Dependency

      TypeBase (5718838)Merge (441a66d)Diff
      classes

      AppCheckTokenProvider
      DataConnect
      DataConnectError
      FirebaseAuthProvider
      MutationManager
      QueryManager
      RESTTransport

      AppCheckTokenProvider
      DataConnect
      DataConnectError
      DataConnectOperationError
      FirebaseAuthProvider
      MutationManager
      QueryManager
      RESTTransport

      + DataConnectOperationError

    • terminate

      Size

      TypeBase (5718838)Merge (441a66d)Diff
      size12.4 kB12.6 kB+181 B (+1.5%)
      size-with-ext-deps30.7 kB30.8 kB+183 B (+0.6%)

      Dependency

      TypeBase (5718838)Merge (441a66d)Diff
      classes

      AppCheckTokenProvider
      DataConnect
      DataConnectError
      FirebaseAuthProvider
      MutationManager
      QueryManager
      RESTTransport

      AppCheckTokenProvider
      DataConnect
      DataConnectError
      DataConnectOperationError
      FirebaseAuthProvider
      MutationManager
      QueryManager
      RESTTransport

      + DataConnectOperationError

    • toQueryRef

      Size

      TypeBase (5718838)Merge (441a66d)Diff
      size13.3 kB13.5 kB+181 B (+1.4%)
      size-with-ext-deps38.6 kB38.8 kB+183 B (+0.5%)

      Dependency

      TypeBase (5718838)Merge (441a66d)Diff
      classes

      AppCheckTokenProvider
      DataConnect
      DataConnectError
      FirebaseAuthProvider
      MutationManager
      QueryManager
      RESTTransport

      AppCheckTokenProvider
      DataConnect
      DataConnectError
      DataConnectOperationError
      FirebaseAuthProvider
      MutationManager
      QueryManager
      RESTTransport

      + DataConnectOperationError

    • validateArgs

      Size

      TypeBase (5718838)Merge (441a66d)Diff
      size13.2 kB13.4 kB+181 B (+1.4%)
      size-with-ext-deps38.6 kB38.8 kB+183 B (+0.5%)

      Dependency

      TypeBase (5718838)Merge (441a66d)Diff
      classes

      AppCheckTokenProvider
      DataConnect
      DataConnectError
      FirebaseAuthProvider
      MutationManager
      QueryManager
      RESTTransport

      AppCheckTokenProvider
      DataConnect
      DataConnectError
      DataConnectOperationError
      FirebaseAuthProvider
      MutationManager
      QueryManager
      RESTTransport

      + DataConnectOperationError

    • validateDCOptions

      Size

      TypeBase (5718838)Merge (441a66d)Diff
      size12.6 kB12.8 kB+181 B (+1.4%)
      size-with-ext-deps30.9 kB31.0 kB+183 B (+0.6%)

      Dependency

      TypeBase (5718838)Merge (441a66d)Diff
      classes

      AppCheckTokenProvider
      DataConnect
      DataConnectError
      FirebaseAuthProvider
      MutationManager
      QueryManager
      RESTTransport

      AppCheckTokenProvider
      DataConnect
      DataConnectError
      DataConnectOperationError
      FirebaseAuthProvider
      MutationManager
      QueryManager
      RESTTransport

      + DataConnectOperationError

Test Logs

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

@maneesht maneesht marked this pull request as ready for review March 19, 2025 21:56
@maneesht maneesht requested review from aashishpatil-g and a team as code owners March 19, 2025 21:56
@maneesht maneesht requested review from a team as code owners March 19, 2025 21:57
Copy link
Contributor

@stephenarosaj stephenarosaj left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@dconeybe dconeybe left a comment

Choose a reason for hiding this comment

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

Looks great!

@maneesht maneesht merged commit edb4001 into main Mar 21, 2025
47 of 48 checks passed
@maneesht maneesht deleted the mtewani/partial-errors branch March 21, 2025 22:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants