🤖 Merge PR DefinitelyTyped#73353 fix(node-pushnotifications): fix fcm settings definition by @vitalyster

Author: vitalyster

types/node-pushnotifications/index.d.ts

@@ -1,7 +1,7 @@
 /// <reference types="node" />
 
+import { Credential, ServiceAccount } from "firebase-admin/app";
 import * as webPush from "web-push";
-
 export = PushNotifications;
 
 declare class PushNotifications {
@@ -34,8 +34,10 @@ declare namespace PushNotifications {
         } | undefined;
         /** Firebase Cloud Messaging  */
         fcm?: {
-            /** FCM token */
-            id?: string | undefined;
+            /** Firebase settings */
+            appName: string;
+            serviceAccountKey: ServiceAccount;
+            credential: Credential | null;
         } | undefined;
         /** Apple Push Notifications */
         apn?: {

types/node-pushnotifications/node-pushnotifications-tests.ts

@@ -1,12 +1,14 @@
 import PushNotifications = require("node-pushnotifications");
 import { supportedContentEncodings } from "web-push";
 
-const settings = {
+const settings: PushNotifications.Settings = {
     gcm: {
         id: "null",
     },
     fcm: {
-        id: "null",
+        appName: "null",
+        serviceAccountKey: require("path/to/serviceAccountKey.json"),
+        credential: null,
     },
     apn: {
         token: {

types/node-pushnotifications/package.json

@@ -7,7 +7,8 @@
     ],
     "dependencies": {
         "@types/node": "*",
-        "@types/web-push": "*"
+        "@types/web-push": "*",
+        "firebase-admin": "^11.5.0"
     },
     "devDependencies": {
         "@types/node-pushnotifications": "workspace:."