Skip to content

feat: add wallet_connected event tracking in Analytics#423

Merged
henyaGreenberg merged 9 commits into
mainfrom
connectWalletEvent
May 24, 2026
Merged

feat: add wallet_connected event tracking in Analytics#423
henyaGreenberg merged 9 commits into
mainfrom
connectWalletEvent

Conversation

@henyaGreenberg

@henyaGreenberg henyaGreenberg commented Dec 28, 2025

Copy link
Copy Markdown
Contributor

create event in the data for wallet connect
https://github.com/Bring-Web3-LTD/TODO/issues/92

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR adds tracking for wallet connection events in Google Analytics. When a user's wallet address changes, a wallet_connected event is now sent to both the backend analytics service and Google Analytics with details about the previous and current wallet addresses.

Key Changes:

  • Added wallet_connected to the list of trackable events
  • Implemented a useEffect hook to monitor wallet address changes and trigger analytics events
  • Refactored sendGaEvent from a regular function to a useCallback hook to enable its use in other hooks

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread iframe-frontend/src/context/googleAnalyticsContext.tsx
Comment thread iframe-frontend/src/context/googleAnalyticsContext.tsx Outdated
Comment thread iframe-frontend/src/context/googleAnalyticsContext.tsx Outdated

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread iframe-frontend/src/context/googleAnalyticsContext.tsx Outdated
Comment thread iframe-frontend/src/context/googleAnalyticsContext.tsx Outdated

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread iframe-frontend/src/context/googleAnalyticsContext.tsx Outdated
Comment thread iframe-frontend/src/context/googleAnalyticsContext.tsx Outdated
@henyaGreenberg henyaGreenberg changed the base branch from dev to main February 3, 2026 11:44
Comment thread iframe-frontend/src/context/googleAnalyticsContext.tsx Outdated
@NoamMK NoamMK changed the title feat: add wallet_connected event tracking in Google Analytics and bac… feat: add wallet_connected event tracking in Analytics Feb 25, 2026
kalfada
kalfada previously approved these changes Mar 14, 2026

@kalfada kalfada left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

AyalaGarbuz
AyalaGarbuz previously approved these changes Mar 16, 2026

@AyalaGarbuz AyalaGarbuz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

NC

# Please enter a commit message to explain why this merge is necessary,
# especially if it merges an updated upstream into a topic branch.
#
# Lines starting with '#' will be ignored, and an empty message aborts
# the commit.
@henyaGreenberg henyaGreenberg dismissed stale reviews from kalfada and AyalaGarbuz via b20b464 May 20, 2026 10:12
@kalfada kalfada requested review from NoamMK and kalfada May 20, 2026 10:14

@kalfada kalfada left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@NoamMK NoamMK left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

See comment (It's not a blokcer)

Comment thread iframe-frontend/src/context/googleAnalyticsContext.tsx
@henyaGreenberg henyaGreenberg merged commit 95f4bbf into main May 24, 2026
1 check 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.

5 participants