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): Recommend fee #457

Merged
merged 9 commits into from
Dec 21, 2022
Merged

feat(bridge-ui): Recommend fee #457

merged 9 commits into from
Dec 21, 2022

Conversation

cyberhorsey
Copy link
Contributor

No description provided.

@codecov
Copy link

codecov bot commented Dec 19, 2022

Codecov Report

Merging #457 (d90f69a) into main (d5532f6) will increase coverage by 0.30%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #457      +/-   ##
==========================================
+ Coverage   66.48%   66.79%   +0.30%     
==========================================
  Files         102      107       +5     
  Lines        2787     2831      +44     
  Branches      328      333       +5     
==========================================
+ Hits         1853     1891      +38     
- Misses        860      867       +7     
+ Partials       74       73       -1     
Flag Coverage Δ *Carryforward flag
bridge-ui 99.25% <100.00%> (+0.15%) ⬆️
protocol 58.54% <ø> (-0.43%) ⬇️ Carriedforward from 4478fb4
relayer 68.92% <ø> (ø) Carriedforward from 4478fb4
ui 100.00% <100.00%> (ø) Carriedforward from 4478fb4

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

Impacted Files Coverage Δ
packages/bridge-ui/src/erc20/bridge.ts 100.00% <100.00%> (ø)
packages/bridge-ui/src/eth/bridge.ts 100.00% <100.00%> (ø)
packages/bridge-ui/src/proof/service.ts 100.00% <100.00%> (ø)
...ages/bridge-ui/src/utils/recommendProcessingFee.ts 100.00% <100.00%> (ø)
...es/protocol/contracts/thirdparty/LibMerkleTrie.sol 83.13% <0.00%> (-7.23%) ⬇️
packages/bridge-ui/src/store/providers.ts 100.00% <0.00%> (ø)
packages/bridge-ui/src/constants/abi/HeaderSync.ts 100.00% <0.00%> (ø)
packages/bridge-ui/src/domain/fee.ts 100.00% <0.00%> (ø)
packages/bridge-ui/src/store/signer.ts 100.00% <0.00%> (ø)

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

@cyberhorsey cyberhorsey changed the title feat(bridge-ui) Recommend fee feat(bridge-ui): Recommend fee Dec 19, 2022
dantaik
dantaik previously approved these changes Dec 20, 2022
@vercel
Copy link

vercel bot commented Dec 20, 2022

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

Name Status Preview Updated
website 🔄 Building (Inspect) Dec 20, 2022 at 5:03AM (UTC)

d1onys1us
d1onys1us previously approved these changes Dec 20, 2022
@dantaik dantaik enabled auto-merge (squash) December 21, 2022 08:15
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