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

Implement the Ethereum token whitelist logic in Namada #1755

Merged
merged 23 commits into from
Sep 6, 2023

Conversation

sug0
Copy link
Collaborator

@sug0 sug0 commented Jul 25, 2023

Describe your changes

Related to #1290 and anoma/ethereum-bridge#60

Supersedes #1718 (reason: it was running an old CI)

Indicate on which release or other PRs this topic is based on

v0.21.1

Checklist before merging to draft

  • I have added a changelog
  • Git history is in acceptable state

batconjurer
batconjurer previously approved these changes Jul 26, 2023
apps/Cargo.toml Outdated Show resolved Hide resolved
tzemanovic
tzemanovic previously approved these changes Jul 28, 2023
@sug0 sug0 dismissed stale reviews from tzemanovic and batconjurer via ae650c0 July 31, 2023 12:12
@sug0 sug0 force-pushed the tiago/token-whitelist branch from 4fefd3a to ae650c0 Compare July 31, 2023 12:12
@sug0 sug0 requested a review from tzemanovic July 31, 2023 12:41
tzemanovic
tzemanovic previously approved these changes Jul 31, 2023
@sug0 sug0 marked this pull request as draft August 28, 2023 08:24
@sug0 sug0 force-pushed the tiago/token-whitelist branch from 06d3a3d to 20044d0 Compare August 28, 2023 14:43
@sug0 sug0 marked this pull request as ready for review August 28, 2023 14:44
@sug0 sug0 mentioned this pull request Aug 28, 2023
sug0 added a commit that referenced this pull request Aug 30, 2023
sug0 added a commit that referenced this pull request Aug 30, 2023
Fraccaman added a commit that referenced this pull request Sep 6, 2023
* origin/tiago/token-whitelist:
  Add changelog entry for the Ethereum token whitelist
  Move ethbridge-rs deps to the workspace
  Update ethbridge-rs to v0.21.0
  Add whitelisted ERC20 tokens to the genesis file
  Remove whitelist Ethereum events from the ledger
  Update ethbridge-rs to v0.20.0
  Add NUT cmdline flag for Bridge pool transfers
  Test NUTs in the Bridge pool
  Check NUT escrow in Bridge pool VP
  Test burning NUTs
  Rework DAI minting unit tests to account for NUTs
  Fix refunding and burning of NUTs
  Fix unit tests requiring token whitelist checks
  Add helper fn to whitelist tokens in tests
  Mint NUTs upon receiving non-whitelisted assets or exceeding token caps
  Add transfer to Ethereum kinds (ERC20 or NUT)
  Update ethbridge-rs to v0.19.0
  Trigger the NUT native VP
  Test NUT VP
  Add a VP for NUT transfers
  Add non-usable tokens for Ethereum bridge protocol bookkeeping
  Add Eth whitelist key tests
  Add Eth whitelist storage keys
Fraccaman added a commit that referenced this pull request Sep 6, 2023
* origin/tiago/token-whitelist:
  Add changelog entry for the Ethereum token whitelist
  Move ethbridge-rs deps to the workspace
  Update ethbridge-rs to v0.21.0
  Add whitelisted ERC20 tokens to the genesis file
  Remove whitelist Ethereum events from the ledger
  Update ethbridge-rs to v0.20.0
  Add NUT cmdline flag for Bridge pool transfers
  Test NUTs in the Bridge pool
  Check NUT escrow in Bridge pool VP
  Test burning NUTs
  Rework DAI minting unit tests to account for NUTs
  Fix refunding and burning of NUTs
  Fix unit tests requiring token whitelist checks
  Add helper fn to whitelist tokens in tests
  Mint NUTs upon receiving non-whitelisted assets or exceeding token caps
  Add transfer to Ethereum kinds (ERC20 or NUT)
  Update ethbridge-rs to v0.19.0
  Trigger the NUT native VP
  Test NUT VP
  Add a VP for NUT transfers
  Add non-usable tokens for Ethereum bridge protocol bookkeeping
  Add Eth whitelist key tests
  Add Eth whitelist storage keys
@Fraccaman Fraccaman merged commit 1640774 into main Sep 6, 2023
@Fraccaman Fraccaman deleted the tiago/token-whitelist branch September 6, 2023 21:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants