firebase / firebase/firebase-admin-node

[FR] Add test for "must not exceed the maximum of 4 weeks"

オープン
#2,832 コメント 1 件 リアクション 0 件 担当者 1 名 @chong-shao が担当を希望しています GitHub で見る
api: messaging type: feature request
主要言語
TypeScript
スター
1.7k
フォーク
419
平均マージ
3日 10時間
マージ済み PR(30日)
16

説明

**Is your feature request related to a problem? Please describe.**
When inserting a pushnotification, its TTL may not exceed 4 weeks according to docs.
This is only checked after the request is send to the API. It should be checked in the library, to easier catch bugs and not annoy the API.

**Describe the solution you'd like**
Check the payload TTL locally (in the library) for common mistakes.

**Describe alternatives you've considered**
Right now its done to the enduser of checking this. And the results may vary. In the end it can happend due to bugs or other things that invalid payloads still get send to the API - when this library could've prevented that.

Thanks for consideration.

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

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

評価

この issue はまだ評価されていません。

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

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