Skip to content

added firebase basic setup and analytics#6

Merged
poocart merged 1 commit intomainfrom
feature/firebase
Nov 15, 2023
Merged

added firebase basic setup and analytics#6
poocart merged 1 commit intomainfrom
feature/firebase

Conversation

@poocart
Copy link
Contributor

@poocart poocart commented Nov 15, 2023

Description

  • Adds firebase initial setup and firebase analytics.

How Has This Been Tested?

  • Tested events locally.

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

@poocart poocart requested a review from IAmKio November 15, 2023 12:39
useEffect(() => {
const callback = () => {
setIsPwaInstalled(true);
logEvent('pwa_install_successful');
Copy link
Collaborator

Choose a reason for hiding this comment

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

Very nice, thank you

Copy link
Collaborator

@IAmKio IAmKio left a comment

Choose a reason for hiding this comment

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

LGTM 🔥

@poocart poocart merged commit 2b9badd into main Nov 15, 2023
@poocart poocart deleted the feature/firebase branch November 16, 2023 08:35
IAmKio pushed a commit that referenced this pull request Oct 7, 2024
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