firebase / firebase/firebase-functions

Firebase Crashlytics - Cannot use dynamic params for declaring appId

Open
#1,597 2 comments 2 reactions 0 assignees View on GitHub
api: crashlytics feature request
Dominant language
TypeScript
Stars
1.1k
Forks
232
Avg merge
20h 46m
Merged PRs (30d)
15

Description

### Related issues

I couldn't find any related issues.

### Version info

**node:** 20

**firebase-functions:** 5.0.0

**firebase-tools:** 13.15.2

### Test case

```typescript

import {defineString} from "firebase-functions/params";
import {onVelocityAlertPublished} from "firebase-functions/v2/alerts/crashlytics";
import {logger} from "firebase-functions";

const APP_ID = defineString('APP_ID');

export const detectVelocityAlert = onVelocityAlertPublished({
appId: APP_ID,
}, async (event) => {
logger.log("New velocity alert spotted!");
});
```

### Steps to reproduce

1. Create a Cloud Function using onVelocityAlertPublished.
2. Define the appId using defineString('APP_ID') from firebase-functions/params.
3. Attempt to deploy or run the function.

### Expected behavior

I expect the function to accept a parameter defined via defineString('APP_ID') for the appId in the onVelocityAlertPublished function configuration.

### Actual behavior

When I try to use defineString('APP_ID') to set the appId, the function fails to deploy because it appears to only accept a string literal, not a StringParams.

### Were you able to successfully deploy your functions?

No, the deployment fails.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.