firebase / firebase/firebase-admin-node
onDocumentUpdate event is typed as having data potentially undefined
オープン
- 主要言語
- TypeScript
- スター
- 1.7k
- フォーク
- 419
- 平均マージ
- 3日 10時間
- マージ済み PR(30日)
- 16
説明
It seems weird that an update event could contain nothing, if that's the case an update event did not occur, did it ?
```ts
// firestore.d.ts
export declare function onDocumentUpdated(document: Document, handler: (event: FirestoreEvent | undefined, ParamsOf>) => any | Promise): CloudFunction | undefined, ParamsOf>>;
```
It is annoying because it forces an additional, seemingly unnecessary, check.
コントリビューションガイド
調査の方向性
Start with the firestore.d.ts declaration shown in the issue and inspect the onDocumentUpdated event type. Confirm the intended non-optional event type and verify that the generated declaration no longer requires an unnecessary undefined check.
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- firebase, typescript
- 領域
- api, backend
- issue の種類
- バグ
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 活発さ
- 停滞
- 明瞭さ
- 明確に書かれている
- 初心者へのやさしさ
- 45/100