fix(afs/typings): valueChanges should return Observable<T|null> (angu…

…lar#1321)
myspivey · Apr 10, 2018 · 37ed0d2 · 37ed0d2
1 parent 60b454e
commit 37ed0d2
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/firestore/document/document.ts b/src/firestore/document/document.ts
@@ -89,9 +89,9 @@ export class AngularFirestoreDocument<T> {
   /**
    * Listen to unwrapped snapshot updates from the document.
    */
-  valueChanges(): Observable<T> {
+  valueChanges(): Observable<T|null> {
     return this.snapshotChanges().map(action => {
-      return (action.payload.exists ? action.payload.data() : null) as T;
+      return action.payload.exists ? action.payload.data() as T : null;
     });
   }
 }