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

feat: report walletconnect pairing errors in mixpanel #6360

Merged
merged 2 commits into from
Mar 5, 2024

Conversation

gomesalexandre
Copy link
Contributor

@gomesalexandre gomesalexandre commented Mar 5, 2024

Description

Does what it says on the box

Pull Request Type

  • πŸ› Bug fix (Non-breaking Change: Fixes an issue)
  • πŸ› οΈ Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
  • πŸ’… New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

Risk

High Risk PRs Require 2 approvals

None, only catches and logs the error

What protocols, transaction types or contract interactions might be affected by this PR?

Testing

None

Engineering

Operations

Screenshots (if applicable)

Copy link
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @gomesalexandre and the rest of your teammates on Graphite Graphite

@gomesalexandre gomesalexandre marked this pull request as ready for review March 5, 2024 22:51
@gomesalexandre gomesalexandre requested a review from a team as a code owner March 5, 2024 22:51
@gomesalexandre gomesalexandre enabled auto-merge (squash) March 5, 2024 22:52
@gomesalexandre gomesalexandre merged commit 6b78e31 into develop Mar 5, 2024
4 checks passed
@gomesalexandre gomesalexandre deleted the feat_mixpanel_wc_log branch March 5, 2024 22:57
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