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

[FIX] Wallet Connected Events #1069

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Conversation

flavioislima
Copy link
Contributor

@flavioislima flavioislima commented Sep 12, 2024

Looking at current chart of events we can see a decreasing number of 'Wallet Connected' events which impacts some of our conversion charts. This started around HyperPlay v0.17.

I guess the issue is because we don't call tracking method with this event anywhere in the code so this PR should fix it.

Let me know if this is enough or we should also send other events or parameters.


Use the following Checklist if you have changed something on the Backend or Frontend:

  • Tested the feature and it's working on a current and clean install.
  • Tested the main App features and they are still working on a current and clean install. (Login, Install, Play, Uninstall, Move games, etc.)
  • Created / Updated Tests (If necessary)
  • Created / Updated documentation (If necessary)

@flavioislima flavioislima self-assigned this Sep 12, 2024
@flavioislima flavioislima added the PR: Ready-For-Review PR is ready to be reviewed by peers label Sep 12, 2024
@flavioislima flavioislima marked this pull request as draft October 16, 2024 14:37
@flavioislima flavioislima removed the PR: Ready-For-Review PR is ready to be reviewed by peers label Oct 16, 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