Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update cloud functions to 2nd generation #301

Merged
merged 16 commits into from
Aug 27, 2024

Conversation

Alona-Kirichenko
Copy link
Collaborator

@Alona-Kirichenko Alona-Kirichenko commented Jul 4, 2024

Updating Firebase cloud function from first generation to second generation.
Right now they are deployed only to dev. If you open cloud console you still can see both gen functions of function at dev. After this PR approved I will delete first gen functions from cloud.

Now we need to use 'dotenv' library to configure your environment
Here is more info https://firebase.google.com/docs/functions/config-env?gen=2nd

onCall

onCall, you need to path the whole url and not just a name of the function as we use to do with first gen functions
here is more info invertase/react-native-firebase#6622 (comment)

I deployd almost all functions to europe-north1 region
But

onSchedule

But onSchrdul function doesn't support all region firebase/firebase-functions#1293 (comment)

I didn't change deleteUser to second gen, because we don't use this function anywhere for now. I was commenting it away in case we want to start to use it. We can discuss it

downloadData I will update to second gen in another branch

assignAdminClaim I deleted. It was a function for developers to huck claims. To give to specific user role in clams. I don't think we need it

covetous
covetous previously approved these changes Aug 27, 2024
@Alona-Kirichenko Alona-Kirichenko merged commit 0fcdb33 into main Aug 27, 2024
1 check passed
@Alona-Kirichenko Alona-Kirichenko deleted the update_cloud_functions_to_2nd_generation branch August 27, 2024 13:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants