firebase/functions-samples

Update fcm-notification sample to use latest API instead of legacy API

Open

#410 opened on 2018年5月25日

GitHub で見る
 (3 comments) (0 reactions) (0 assignees)JavaScript (11,901 stars) (3,957 forks)batch import
help wanted

説明

How to reproduce these conditions

Sample name or URL where you found the bug fcm-notification

Failing Function code used (including require/import commands at the top) Update fcm-notification sample to use latest API instead of legacy API with reference to https://stackoverflow.com/questions/50505274/why-use-a-legacy-api-for-a-firebase-fcm-notifications-sample/50510755#50510755

Steps to set up and reproduce Line 74 of https://github.com/firebase/functions-samples/blob/master/fcm-notifications/functions/index.js

コントリビューターガイド

技術スタック
javascriptnodejs
領域
backendcloud
Issue 種別
refactor
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3 hours
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
stale
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
clear
前提条件
Node.jsFirebase FunctionsFCM API
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
70
調査方針
Review the current implementation in functions/index.js line 74, and the Stack Overflow discussion linked in the issue. Then update the code to use the latest Firebase Admin SDK FCM API, likely replacing the legacy sendToDevice method. Check the comments for any additional context or maintainer guidance.
Update fcm-notification sample to use latest API instead of legacy API · firebase/functions-samples#410 | Good First Issue