Skip to content
This repository was 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 <lastperson@gmail.com>
Co-authored-by: Kirill <pisarevkir@gmail.com>
@waymobetta @P1sar
use updated geth flags (ChainSafe#511)
65dc0bb 
Co-authored-by: Kirill <pisarevkir@gmail.com>
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