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): bridge transactions #411

Merged
merged 52 commits into from
Dec 15, 2022
Merged

feat(bridge): bridge transactions #411

merged 52 commits into from
Dec 15, 2022

Conversation

shadab-taiko
Copy link
Contributor

@shadab-taiko shadab-taiko commented Dec 9, 2022

closes #329
closes #327
closes #423

@codecov
Copy link

codecov bot commented Dec 9, 2022

Codecov Report

Merging #411 (e46c04f) into main (e9fda8b) will increase coverage by 0.69%.
The diff coverage is 86.20%.

@@            Coverage Diff             @@
##             main     #411      +/-   ##
==========================================
+ Coverage   65.43%   66.12%   +0.69%     
==========================================
  Files         100      102       +2     
  Lines        2699     2787      +88     
  Branches      313      328      +15     
==========================================
+ Hits         1766     1843      +77     
- Misses        861      871      +10     
- Partials       72       73       +1     
Flag Coverage Δ *Carryforward flag
bridge-ui 99.10% <97.29%> (-0.90%) ⬇️
protocol 58.25% <ø> (ø)
relayer 68.92% <65.00%> (-0.11%) ⬇️
ui 100.00% <100.00%> (ø) Carriedforward from 527a0c5

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

Impacted Files Coverage Δ
packages/relayer/indexer/handle_event.go 51.51% <0.00%> (-0.80%) ⬇️
packages/relayer/message/process_message.go 59.00% <42.10%> (-3.93%) ⬇️
packages/bridge-ui/src/domain/token.ts 80.00% <66.66%> (ø)
packages/relayer/message/is_profitable.go 82.35% <87.50%> (+6.35%) ⬆️
packages/bridge-ui/src/domain/chain.ts 100.00% <100.00%> (ø)
packages/bridge-ui/src/domain/message.ts 100.00% <100.00%> (ø)
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%> (ø)
packages/bridge-ui/src/storage/service.ts 100.00% <100.00%> (ø)
... and 7 more

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

cyberhorsey
cyberhorsey previously approved these changes Dec 15, 2022
d1onys1us
d1onys1us previously approved these changes Dec 15, 2022
@d1onys1us
Copy link
Contributor

@cyberhorsey theres a small merge conflict

@cyberhorsey cyberhorsey enabled auto-merge (squash) December 15, 2022 20:02
packages/bridge-ui/src/App.svelte Show resolved Hide resolved
packages/bridge-ui/src/erc20/bridge.spec.ts Show resolved Hide resolved
packages/bridge-ui/src/erc20/bridge.ts Show resolved Hide resolved
packages/bridge-ui/src/proof/service.spec.ts Show resolved Hide resolved
@cyberhorsey cyberhorsey merged commit 19dd7ab into main Dec 15, 2022
@cyberhorsey cyberhorsey deleted the bridge-transactions branch December 15, 2022 21:57
@gitpoap-bot
Copy link

gitpoap-bot bot commented Dec 15, 2022

Congrats, your important contribution to this open-source project has earned you a GitPOAP!

GitPOAP: 2022 Taiko Contributor:

GitPOAP: 2022 Taiko Contributor GitPOAP Badge

Head to gitpoap.io & connect your GitHub account to mint!

Learn more about GitPOAPs here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants