Skip to content

Conversation

@leoortizz
Copy link
Member

Description

Suggestion to mention parameterized configuration docs from Cloud Functions in Web Frameworks, due to many issues raised before with users that did not know how to configure different .env files when deploying to different projects/environments.

For reference, see: #7631, #8543, #8471

Scenarios Tested

Sample Commands

Copy link
Contributor

@egilmorez egilmorez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry for the delay (pls don't hesitate to ping internally)!

Is there any existing content we could link to? Is this close to the desired guidance: https://firebase.google.com/docs/functions/config-env?gen=1st#deploying_multiple_sets_of_environment_variables_2?

This sentence feels like it needs a little unpacking with an example, but if a link can help, that would work too.

@egilmorez
Copy link
Contributor

- add example
- adjust cloud functions documentation link
@leoortizz leoortizz requested a review from egilmorez July 11, 2025 00:04
@leoortizz
Copy link
Member Author

Sorry for the delay (pls don't hesitate to ping internally)!

Is there any existing content we could link to? Is this close to the desired guidance: https://firebase.google.com/docs/functions/config-env?gen=1st#deploying_multiple_sets_of_environment_variables_2?

This sentence feels like it needs a little unpacking with an example, but if a link can help, that would work too.

thanks @egilmorez! I made a few changes based on your feedback, let me know what you think!

Copy link
Contributor

@egilmorez egilmorez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Leo! One more thing to look at.

@egilmorez egilmorez requested a review from kevinthecheung July 21, 2025 16:12
Copy link
Contributor

@egilmorez egilmorez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LG, thanks!

@leoortizz leoortizz merged commit a6188c3 into master Jul 21, 2025
48 of 50 checks passed
@leoortizz leoortizz deleted the leoortizz_wfParameterizedConfigDocs branch July 21, 2025 19:14
@github-project-automation github-project-automation bot moved this from Approved [PR] to Done in [Cloud] Extensions + Functions Jul 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants