angular / angular/angularfire

Errors from methods returning a Promise are not bubbling up when Promise is converted to Observable

オープン
#3,478 コメント 2 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
TypeScript
スター
7.8k
フォーク
2.2k
平均マージ
22時間 28分
マージ済み PR(30日)
6

説明

In order to achieve consistency, i want to convert methods returning Promises to Observable.
However, errors are ignored when trying to catch the error either from the method or from the caller of the method.

See the code below.

```typescript
createList(newList: NewListDTO) {
const collectionRef = collection(this.db, `lists`);

const newListDoc = addDoc(collectionRef, {
name: newList.name,
uid: newList.uid,
created_at: serverTimestamp(),
});

// return defer(async () => {
// return await newListDoc;
// });

return scheduled(newListDoc, asyncScheduler).pipe(
// FIXME: error is not being caught from firstore addDoc
catchError((err) => {
return throwError(() => err);
}),
);
}
```

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

createList の例から始め、コメントアウトされている defer(async ...) アプローチと scheduled(newListDoc, asyncScheduler)、およびその catchError ハンドラーを比較します。失敗する addDoc の経路を再現し、Promise からのエラーがメソッドとその呼び出し元の両方に届くか確認します。報告された Firestore エラーが返された Observable を通じて伝播すれば完了です。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
firebase, typescript
領域
database
issue の種類
バグ
難易度
3/5
見積もり時間
1〜2日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
35/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。