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] Expose error in LoadBundleTaskProgress #10455

Closed
wants to merge 1 commit into from

Conversation

andrewheard
Copy link
Contributor

Propagates errors, if any, from the underlying LoadBundleTaskProgress C++ type to FIRLoadBundleTaskProgress in the ObjC layer. This allows callers to inspect the type of error that occurred when observing bundle loading progress using the addObserver: API in FIRLoadBundleTask.

In its current form, callers can check whether an error occurred by looking at FIRLoadBundleTaskProgress's state but cannot retrieve any details about the error. However, error details are already exposed when using the loadBundle:completion: API in FIRFirestore.

Needs API review.

Propagates errors, if any, from the underlying LoadBundleTaskProgress C++ type to the FIRLoadBundleTaskProgress in the ObjC layer. This allows callers to inspect the type of error that occurred when observing bundle loading progress using the addObserver API in FIRLoadBundleTask.

Needs API review.
@paulb777
Copy link
Member

Please close, merge, or comment. We plan to close stale PRs on November 28, 2023.

@andrewheard andrewheard deleted the ah-firestore-load-bundle-error branch November 14, 2023 15:58
@firebase firebase locked and limited conversation to collaborators Dec 15, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants