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

fix double initialistation of firebase when creating storage service #810

Merged
merged 1 commit into from
Oct 19, 2024

Conversation

choden-dev
Copy link
Member

@choden-dev choden-dev commented Oct 19, 2024

same issue as vercel/next.js#11351, however we just need to not call getStorage raw without passing in the firebase app

This was causing the admin/events build to sometimes fail

@choden-dev choden-dev changed the title fix double initialistation fix double initialistation of firebase when creating storage service Oct 19, 2024
Copy link

Visit the preview URL for this PR (updated for commit 82adaed):

https://uasc-ceebc--pr810-fix-firebase-build-e-q9ocjhbv.web.app

(expires Sat, 26 Oct 2024 00:33:23 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 36296ceaed1d43e92e7d5e81a72a7bd987560bfa

@choden-dev choden-dev requested a review from asun555 October 19, 2024 00:33
@zlrkw11
Copy link
Member

zlrkw11 commented Oct 19, 2024

lookin crisp

Copy link
Member

@zlrkw11 zlrkw11 left a comment

Choose a reason for hiding this comment

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

clean

@choden-dev choden-dev requested a review from zlrkw11 October 19, 2024 01:12
@choden-dev choden-dev merged commit d17d321 into master Oct 19, 2024
10 checks passed
@choden-dev choden-dev deleted the fix-firebase-build-error branch October 19, 2024 01:13
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