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

Filter 'Talisman not configured' error from Sentry #786

Merged
merged 1 commit into from
Jan 23, 2025

Conversation

gndelia
Copy link
Contributor

@gndelia gndelia commented Jan 23, 2025

Description

This PR filters out the following error from Sentry:

Error: Talisman extension has not been configured yet. Please continue with onboarding.

As

  1. Talisman is not supported by the portal
  2. This is an error on user land, as it seems they haven't configured the wallet properly.

See docs for filtering events on sentry config.

I will archive this error on Sentry UI.

I also re-ordered the error messages so they are sorted alphabetically

Screenshots

No visible changes to users.

Related issue(s)

Related to #715

Checklist

  • Manual testing passed.
  • Automated tests added, or N/A.
  • Documentation updated, or N/A.
  • Environment variables set in CI, or N/A.

@gndelia gndelia merged commit cfe5d54 into main Jan 23, 2025
6 checks passed
@gndelia gndelia deleted the filter-out-talisman-not-configured branch January 23, 2025 21:11
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.

3 participants