Skip to content
This repository has been archived by the owner on May 9, 2024. It is now read-only.

update Deposit ETH #560

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

update Deposit ETH #560

wants to merge 5 commits into from

Conversation

Fankouzu
Copy link

Description

Related Issue Or Context

Closes: #

How Has This Been Tested? Testing details.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation

Checklist:

  • I have commented my code, particularly in hard-to-understand areas.
  • I have ensured that all acceptance criteria (or expected behavior) from issue are met
  • I have updated the documentation locally and in chainbridge-docs.
  • I have added tests to cover my changes.
  • I have ensured that all the checks are passing and green, I've signed the CLA bot

Neuti Yoo and others added 5 commits May 9, 2022 19:20
Remove redundant ABIEncoderV2 pragma (ChainSafe#539)
66ba8c4
@viatrix @lastperson @P1sar
Viatrix/fee handler (ChainSafe#519)
963103b 
* WIP: Basic FeeHandler

* WIP: Basic FeeHandler

* Fee handler with oracle

* FeeHandler: fixes after comments

* Fix after review

* Simplify abi decoding

* Added expiration time to oracle data

* sign oracle message by ethers.js

* Remove prefix from signature

* Add FeeCollected event

* Add FeeDistributed event

Co-authored-by: Oleksii Matiiasevych <[email protected]>
Co-authored-by: Kirill <[email protected]>
@waymobetta @P1sar
use updated geth flags (ChainSafe#511)
65dc0bb 
Co-authored-by: Kirill <[email protected]>
Remove ignored visibility for constructor (ChainSafe#552)
6bcfdb1
@Fankouzu
update Deposit ETH
5e2e864
@CLAassistant
Copy link

CLAassistant commented May 31, 2022 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

@xiaohanzhu
Copy link

This pull request is to simplify ETH/native gas token handling for the users. They do not have to wrap ETH when interacting with the bridge

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@Fankouzu @CLAassistant @xiaohanzhu @viatrix @waymobetta