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): fetch transactions from relayer api #13244

Merged
merged 3 commits into from
Mar 4, 2023

Conversation

shadab-taiko
Copy link
Contributor

@shadab-taiko shadab-taiko commented Mar 2, 2023

Addresses #4772 (and duplicate #12976)

@codecov
Copy link

codecov bot commented Mar 2, 2023

Codecov Report

Merging #13244 (9d2806d) into main (de8a9b2) will decrease coverage by 0.04%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##             main   #13244      +/-   ##
==========================================
- Coverage   61.76%   61.72%   -0.04%     
==========================================
  Files         117      117              
  Lines        3452     3454       +2     
  Branches      483      483              
==========================================
  Hits         2132     2132              
- Misses       1236     1238       +2     
  Partials       84       84              
Flag Coverage Δ *Carryforward flag
bridge-ui 93.80% <66.66%> (-0.54%) ⬇️
protocol 52.55% <ø> (ø) Carriedforward from 520fce1
relayer 65.18% <ø> (ø) Carriedforward from 520fce1
ui 100.00% <ø> (ø) Carriedforward from 520fce1

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

Impacted Files Coverage Δ
packages/bridge-ui/src/storage/service.ts 94.91% <33.33%> (-3.34%) ⬇️
packages/bridge-ui/src/domain/chain.ts 87.87% <100.00%> (ø)

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

@cyberhorsey
Copy link
Contributor

Just one comment, otherwise LGTM!

@dantaik dantaik enabled auto-merge March 4, 2023 02:00
@dantaik dantaik added this pull request to the merge queue Mar 4, 2023
Merged via the queue into main with commit 9732cc0 Mar 4, 2023
@dantaik dantaik deleted the transactions-from-relayer branch March 4, 2023 02:00
@github-actions github-actions bot mentioned this pull request Mar 4, 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