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
I142 integrate firebase into the home page #167
I142 integrate firebase into the home page #167
Conversation
…se_into_the_home_page
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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.
Looks pretty good, just a couple notes to handle on the index page and summary component. It's mostly about moving anything specific to the summary out of the index page and reducing the amount of function calls in index.
I'll leave reviewing the triggers to @NicholasChoong
…anged and refactored
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.
I'm happy with all the React/Typescript stuff, nice work 🚀
Thank you for the review @scratchclaggy! |
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.
Great work!! The trigger function works flawlessly.
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.
Nice
@NicholasChoong Thanks for the review 😊 |
All g |
Change Summary
Change Form
Other Information
The trigger only works with the emulator enabled and does not currently work on the live server.
Paid plan will be needed to be able to deploy the functions.
Related Issue