-
Notifications
You must be signed in to change notification settings - Fork 4
Update dependencies and applications #18
base: master
Are you sure you want to change the base?
Conversation
No super, to se mi hodí, zrovna to chci použít :-D |
functions/.gitignore
Outdated
@@ -0,0 +1 @@ | |||
*.local |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Duplicated with .gitignote
rules at root folder.
functions/src/index.ts
Outdated
import dotenv from "dotenv" | ||
|
||
dotenv.config() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you have any .env
config required for production? Pls, put here any sample file and add installation instructions to README.md
file.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe it's whole unnecessary, because Firebase is using dotfile
internally: https://firebase.google.com/docs/functions/config-env#env-variables
public/admin/index.html
Outdated
<script src="firebaseConfig.js"></script> | ||
<script> | ||
// Initialize Firebase | ||
firebase.initializeApp(firebaseConfig); | ||
firebase.auth().useEmulator("http://localhost:9099"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ALERT ALERT, unwanted dev emulator - can cause to broke production!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh, my bad, forgot to remove it
No description provided.