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(bridge-ui): sentry integration #13943

Merged
merged 14 commits into from
Jun 12, 2023
Merged

feat(bridge-ui): sentry integration #13943

merged 14 commits into from
Jun 12, 2023

Conversation

jscriptcoder
Copy link
Contributor

@jscriptcoder jscriptcoder commented Jun 9, 2023

Basic Sentry integration, capturing exceptions for some key functionalities:

  • minting
  • approving tokens
  • bridging
  • claiming (and retrying)
  • releasing
  • calculating recommended fee
  • switching chains
  • connecting wallet

@codecov
Copy link

codecov bot commented Jun 9, 2023

Codecov Report

Merging #13943 (c1c7463) into main (22a1171) will increase coverage by 0.02%.
The diff coverage is 100.00%.

❗ Current head c1c7463 differs from pull request most recent head 5c88a23. Consider uploading reports for the commit 5c88a23 to get more accurate results

@@            Coverage Diff             @@
##             main   #13943      +/-   ##
==========================================
+ Coverage   52.04%   52.07%   +0.02%     
==========================================
  Files         146      146              
  Lines        3808     3810       +2     
  Branches      336      336              
==========================================
+ Hits         1982     1984       +2     
  Misses       1716     1716              
  Partials      110      110              
Flag Coverage Δ *Carryforward flag
bridge-ui 94.44% <100.00%> (+0.01%) ⬆️
eventindexer 78.02% <ø> (ø) Carriedforward from 222ff52
protocol 0.00% <ø> (ø) Carriedforward from 222ff52
relayer 60.14% <ø> (ø) Carriedforward from 222ff52
ui 100.00% <ø> (ø) Carriedforward from 222ff52

*This pull request uses carry forward flags. Click here to find out more.

Impacted Files Coverage Δ
packages/bridge-ui/src/signer/subscriber.ts 100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@dantaik dantaik requested a review from cyberhorsey June 10, 2023 00:32
dantaik
dantaik previously approved these changes Jun 12, 2023
@jscriptcoder jscriptcoder added this pull request to the merge queue Jun 12, 2023
Merged via the queue into main with commit 05baee8 Jun 12, 2023
@jscriptcoder jscriptcoder deleted the integrate_sentry branch June 12, 2023 10:37
@github-actions github-actions bot mentioned this pull request Jun 12, 2023
dantaik pushed a commit that referenced this pull request Jun 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants