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(bridge-ui): add extra info to sentry #13972

Merged
merged 2 commits into from
Jun 12, 2023
Merged

fix(bridge-ui): add extra info to sentry #13972

merged 2 commits into from
Jun 12, 2023

Conversation

jscriptcoder
Copy link
Contributor

add valuable extra info to sentry

@codecov
Copy link

codecov bot commented Jun 12, 2023

Codecov Report

Merging #13972 (4dff082) into main (300be15) will decrease coverage by 0.01%.
The diff coverage is 66.66%.

❗ Current head 4dff082 differs from pull request most recent head f3ebcc2. Consider uploading reports for the commit f3ebcc2 to get more accurate results

@@            Coverage Diff             @@
##             main   #13972      +/-   ##
==========================================
- Coverage   52.07%   52.07%   -0.01%     
==========================================
  Files         146      146              
  Lines        3810     3814       +4     
  Branches      336      336              
==========================================
+ Hits         1984     1986       +2     
- Misses       1716     1718       +2     
  Partials      110      110              
Flag Coverage Δ *Carryforward flag
bridge-ui 94.23% <66.66%> (-0.21%) ⬇️
eventindexer 78.02% <ø> (ø) Carriedforward from 08baec0
protocol 0.00% <ø> (ø) Carriedforward from 08baec0
relayer 60.14% <ø> (ø) Carriedforward from 08baec0
ui 100.00% <ø> (ø) Carriedforward from 08baec0

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

Impacted Files Coverage Δ
packages/protocol/contracts/L1/TaikoL1.sol 0.00% <ø> (ø)
packages/protocol/contracts/L1/TaikoToken.sol 0.00% <ø> (ø)
...ckages/protocol/contracts/L1/libs/LibProposing.sol 0.00% <ø> (ø)
packages/protocol/contracts/L1/libs/LibProving.sol 0.00% <ø> (ø)
packages/protocol/contracts/L1/libs/LibUtils.sol 0.00% <ø> (ø)
...ckages/protocol/contracts/L1/libs/LibVerifying.sol 0.00% <ø> (ø)
packages/protocol/contracts/L2/TaikoL2.sol 0.00% <ø> (ø)
packages/protocol/contracts/bridge/Bridge.sol 0.00% <ø> (ø)
...ackages/protocol/contracts/bridge/BridgedERC20.sol 0.00% <ø> (ø)
packages/protocol/contracts/bridge/EtherVault.sol 0.00% <ø> (ø)
... and 38 more

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

@jscriptcoder jscriptcoder self-assigned this Jun 12, 2023
@jscriptcoder jscriptcoder marked this pull request as ready for review June 12, 2023 13:59
davidtaikocha
davidtaikocha previously approved these changes Jun 12, 2023
@davidtaikocha davidtaikocha enabled auto-merge June 12, 2023 13:59
@davidtaikocha davidtaikocha added this pull request to the merge queue Jun 12, 2023
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