-
Notifications
You must be signed in to change notification settings - Fork 82
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
EVM tokens deposit & withdraw #24
Merged
Merged
Changes from all commits
Commits
Show all changes
38 commits
Select commit
Hold shift + click to select a range
98b302a
Added prover & extended types
mrLSD 97a3759
Mode Borsh args from types to parameters
mrLSD a889328
Merge branch 'master' of github.com:aurora-is-near/aurora-engine into…
mrLSD 25154f0
Added fungible tokens
mrLSD a0942a2
Added eth-connector
mrLSD 5876814
Modify assert for fee
mrLSD facc786
Fix formatting
mrLSD b055a54
Extend eth-conenctor with EVM token logic
mrLSD eb9e361
Changed eth-connector deposit logic
mrLSD fcc1332
Added changes for ETH deposit/withdraw and Engine changes
mrLSD 978474b
Mint ETH-tokens
mrLSD 041879e
Added: transfer_eth
mrLSD b5ffb23
ETH withdraw basic method
mrLSD a7e73f4
eth-withdraw validation structure & modified Deposit-eth fields
mrLSD 45a5ca9
eth transfer and withdraw logic
mrLSD 676a86c
eip712 message verifier - started encoding
mrLSD e345d71
added encode-packed
mrLSD de7f36d
virefy EIP712 message for withdraw
mrLSD f1f6479
Changed EIP712 message fields
mrLSD 944b6e2
Modify logs for EIP712 messages
mrLSD 1201a39
Test EIP712
mrLSD 11dec10
Tests EIP712
mrLSD 89f572c
Integration tests for encode_withdraw_eip712
mrLSD d6830ea
Integration tests for encode_withdraw_eip712
mrLSD dd1f8e1
EIP712-Withdraw: improvements and fixes.
sept-en 96b3c0e
Extend tests for eth-connector
mrLSD 348e5cc
Merge branch 'evm-deposit-withdraw' of github.com:aurora-is-near/auro…
mrLSD 80f033b
eth-connector test deposit & balance & total_supply
mrLSD 07251fd
Imporved tests
mrLSD 2de5af8
FT tests
mrLSD e71d52a
Fixed verify_transfer_eip712
mrLSD 09b79c1
Change test_withdraw_near
mrLSD f2e1f11
Tests for: ft_transfer, ft_transfer_call
mrLSD 356ad42
test_eth_deposit_balance_total_supply
mrLSD 6cae502
test and ifx: deposit_eth, withdraw_near
mrLSD d920662
References in fungible token (#29)
birchmd 27f90ef
Fix: hide logging behind feature flag
birchmd f875d36
Fix: clippy errors in eth-connector (#32)
birchmd File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure what our release pipeline looks like, but if it uses
make release
then I think we should not be including theintegration-test
feature by default. That feature should only be enabled when compiling the contract for testing.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As I said @artob we should remove it from Make file