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: filter out errors #2867

Merged
merged 1 commit into from
Jun 25, 2024
Merged

feat: filter out errors #2867

merged 1 commit into from
Jun 25, 2024

Conversation

therealemjy
Copy link
Member

Changes

  • filter out errors from third-party frames
  • stop recording errors about missing token records. These are clogging Sentry due to some users keeping the dApp open for extended periods of time and don't highlight actual JS errors.

Copy link

vercel bot commented Jun 24, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
mainnet ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 24, 2024 0:57am
preview ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 24, 2024 0:57am
testnet ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 24, 2024 0:57am

Copy link

changeset-bot bot commented Jun 24, 2024

🦋 Changeset detected

Latest commit: dba3a2a

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@venusprotocol/evm Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link
Contributor

Coverage Report for ./apps/evm

Status Category Percentage Covered / Total
🔵 Lines 78.58% 46444 / 59098
🔵 Statements 78.58% 46444 / 59098
🔵 Functions 70.94% 879 / 1239
🔵 Branches 83.02% 3889 / 4684
File Coverage
File Stmts % Branch % Funcs % Lines Uncovered Lines
Changed Files
apps/evm/src/clients/api/queries/getIsolatedPools/index.ts 93.99% 73.8% 100% 93.99% 35-36, 73-74, 77-78, 81-82, 181, 193-194, 201-202, 210-211, 221-222
apps/evm/src/clients/api/queries/getIsolatedPools/formatOutput/formatDistributions.ts 97.16% 77.77% 100% 97.16% 59-60, 65-66
apps/evm/src/clients/api/queries/getIsolatedPools/formatOutput/index.ts 93.91% 71.42% 100% 93.91% 78-79, 90, 135, 177-180, 184-187, 278-283
apps/evm/src/clients/api/queries/getLegacyPool/index.ts 95.79% 65.71% 100% 95.79% 66-67, 70-71, 114, 129-130, 168, 192
apps/evm/src/clients/api/queries/getLegacyPool/formatToPool/index.ts 93.58% 62.71% 100% 93.58% 99-101, 103-104, 116-117, 139, 142-143, 149, 152-153, 159, 162-163, 169, 172-173, 179, 182-183, 238, 243, 316, 323, 333
apps/evm/src/clients/api/queries/getPendingRewards/index.ts 93.25% 60% 100% 93.25% 28-32, 39, 95-96, 135-136, 139-140
apps/evm/src/clients/api/queries/getPendingRewards/formatOutput/formatRewardSummaryData.ts 95.29% 77.77% 100% 95.29% 34-35, 64-65
apps/evm/src/clients/api/queries/getPendingRewards/formatOutput/formatToPrimePendingRewardGroup.ts 92.85% 50% 100% 92.85% 31-32, 38-39, 45-46
apps/evm/src/clients/api/queries/getPendingRewards/formatOutput/formatToVaultPendingRewardGroup.ts 90.32% 50% 100% 90.32% 27-28, 33-34, 47-48
apps/evm/src/clients/api/queries/getPendingRewards/formatOutput/formatToVestingVaultPendingRewardGroup.ts 88.23% 28.57% 100% 88.23% 34-35, 40-41, 49-50, 56-57, 69-70
apps/evm/src/clients/api/queries/getVTokens/index.ts 94.62% 84.61% 100% 94.62% 57-59, 61-62
apps/evm/src/libs/errors/ErrorBoundary/index.tsx 96.15% 100% 0% 96.15% 25
Generated in workflow #8465

@therealemjy therealemjy merged commit 3c0164e into main Jun 25, 2024
6 checks passed
@therealemjy therealemjy deleted the feat/error-filtering branch June 25, 2024 06:56
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.

None yet

2 participants