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

Integrate firebase for deployment #727

Merged
merged 5 commits into from Nov 24, 2022
Merged

Conversation

kachick
Copy link
Member

@kachick kachick commented Nov 24, 2022

@github-actions
Copy link
Contributor

github-actions bot commented Nov 24, 2022

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

https://mobu-waiwai--pr727-replace-netlify-with-o8u359jc.web.app

(expires Thu, 01 Dec 2022 15:11:51 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 295e29f2bd6fa9acc70abd1b84aec0de70cff3f6

Comment on lines +33 to +35
repoToken: '${{ secrets.GITHUB_TOKEN }}'
firebaseServiceAccount: '${{ secrets.FIREBASE_SERVICE_ACCOUNT_MOBU_WAIWAI }}'
projectId: mobu-waiwai
Copy link
Member Author

Choose a reason for hiding this comment

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

Without specified expire, firebase keeps 7 days

Comment on lines +30 to +31
- run: npm run build
- uses: FirebaseExtended/action-hosting-deploy@v0
Copy link
Member Author

Choose a reason for hiding this comment

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

I use https://github.com/kachick/wait-other-jobs in my repos just before deploy steps. However excluded in this PR for a simple first step.

<head>
<meta charset="utf-8" />
<meta name="viewport" content="width=device-width, initial-scale=1" />
<title>Page Not Found</title>
Copy link
Member Author

Choose a reason for hiding this comment

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

This file has been generated by firebase CLI

APP_COMMIT_REF: JSON.stringify(commitRef),
},
export default defineConfig(({ mode }) => {
const env = loadEnv(mode, process.cwd());
Copy link
Member Author

Choose a reason for hiding this comment

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

import.meta can't be used in vite.config.ts

Comment on lines +30 to +31
- name: Specify actual commit ref for revisions
run: echo VITE_COMMIT_REF=${{ github.event.pull_request.head.sha }} > .env
Copy link
Member Author

Choose a reason for hiding this comment

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

firebase actions looks generating special merging commit for internal use. It breaks #444 in staging deploy. So this and https://github.com/mobu-of-the-world/mobu/pull/727/files#r1031619146 is the workaround.

@kachick
Copy link
Member Author

kachick commented Nov 24, 2022

スクリーンショット 2022-11-25 001509

🎉

@kachick kachick requested a review from pankona November 24, 2022 15:15
@kachick
Copy link
Member Author

kachick commented Nov 24, 2022

Talked with @pankona in discord. This feature is mandatory, so temporary I created the firebase project and integrated both 😬

So I'll turn off the netlify integration after merging.

@kachick kachick merged commit 429d537 into main Nov 24, 2022
@kachick kachick deleted the replace-netlify-with-firebase branch November 24, 2022 15:18
kachick added a commit that referenced this pull request Nov 28, 2022
kachick added a commit to mobu-of-the-world/emobu that referenced this pull request Dec 22, 2022
@kachick kachick mentioned this pull request Feb 10, 2023
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.

Manage deployment with code as possible
1 participant