You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Related, currently, the docData helper exported by this library is incorrectly typed (see issue #37) as it actually returns Observable<T | undefined>.
This is a request to update docData to return Observable<T | null>. Returning null rather than undefined when a document doesn't exist would be helpful when saving the output to a variable as it is easier to distinguish between a pending result (undefined) and a known null result. I realize that this would be a breaking change.
Related, currently, the
docData
helper exported by this library is incorrectly typed (see issue #37) as it actually returnsObservable<T | undefined>
.This is a request to update
docData
to returnObservable<T | null>
. Returningnull
rather thanundefined
when a document doesn't exist would be helpful when saving the output to a variable as it is easier to distinguish between a pending result (undefined
) and a knownnull
result. I realize that this would be a breaking change.Example:
The text was updated successfully, but these errors were encountered: