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

firebase init #12

Merged
merged 7 commits into from
Oct 12, 2024
Merged

firebase init #12

merged 7 commits into from
Oct 12, 2024

Conversation

sup3rUman
Copy link
Owner

No description provided.

@sup3rUman sup3rUman requested a review from SammyMusyoki July 11, 2024 23:12

// Your web app's Firebase configuration
// For Firebase JS SDK v7.20.0 and later, measurementId is optional
const firebaseConfig = {
Copy link
Collaborator

Choose a reason for hiding this comment

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

The secret keys should not be pushed to github... They should be in an .env file

Copy link
Owner Author

Choose a reason for hiding this comment

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

Done

Copy link

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

https://px-0-d6b35--pr12-dev-zd9oxkeq.web.app

(expires Thu, 19 Sep 2024 09:47:27 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: caad915502cf758d1b029d1af8b0f6ad4618c1c0

Repository owner deleted a comment from vercel bot Sep 12, 2024
@SammyMusyoki SammyMusyoki merged commit 236fff1 into main Oct 12, 2024
0 of 2 checks passed
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