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

New Vault: ETH native support and arbitrary connectors for managing different token standards #129

Merged
merged 39 commits into from Mar 20, 2018
Merged
Changes from 1 commit
Commits
Show all changes
39 commits
Select commit Hold shift + click to select a range
67071c6
Start work on new vault
izqui Mar 13, 2018
17650ce
Scaffold connectors
izqui Mar 14, 2018
eac5bf4
Add ETHConnector
izqui Mar 14, 2018
d9d0314
ERC20, ERC777, ERC721 connectors
izqui Mar 14, 2018
ba342f9
Add ETH connector fallback function for easy eth deposits
izqui Mar 14, 2018
734d61b
Start implementing automated interface detection
izqui Mar 14, 2018
e09a5a1
Implement token standard detection
izqui Mar 14, 2018
d433367
Publish 1.1.3
izqui Mar 14, 2018
b074337
Merge
izqui Mar 14, 2018
7c56659
Merge branch 'master' into good-vault
izqui Mar 14, 2018
acc1157
Improve style
izqui Mar 14, 2018
d01b101
Get new vault to almost compile
izqui Mar 14, 2018
d5cc2f7
It compiles, note for the future, avoid passing around structs
izqui Mar 14, 2018
270a636
Protect transfer and register standard behind roles
izqui Mar 15, 2018
d43d49a
Scaffold tests
izqui Mar 15, 2018
22844bd
Fix eth fallback
izqui Mar 15, 2018
ea70fd8
Test
izqui Mar 15, 2018
0bca179
finance: Adapt to new Vault
bingen Mar 15, 2018
5b83a0c
Make tests pass
izqui Mar 16, 2018
41d164d
Merge branch 'good-vault-finance-2' of github.com:aragon/aragon-apps …
izqui Mar 16, 2018
7b45dba
vault: deploy connectors on constructor to simplify templates
izqui Mar 16, 2018
e2b17de
Adapt templates to new vault and remove all ethertoken references
izqui Mar 16, 2018
191d312
finance: Adapt to new Vault (#135)
bingen Mar 16, 2018
dd666f6
Merge master
izqui Mar 16, 2018
aba830f
Merge branch 'master' of github.com:aragon/aragon-apps into new-vault…
izqui Mar 16, 2018
835ae33
Merge branch 'good-vault' of github.com:aragon/aragon-apps into good-…
izqui Mar 16, 2018
abf72d7
Update to aOs v3.1.1
izqui Mar 16, 2018
13fc00d
Fix devtemplate
izqui Mar 16, 2018
6117167
Fix lint errors
bingen Mar 16, 2018
b787a5d
vault: fix coverage and tests
bingen Mar 17, 2018
4cf0a69
finance: fix tests
bingen Mar 17, 2018
db70f08
tmp
bingen Mar 16, 2018
ef13ad5
Remove abis file
izqui Mar 17, 2018
e450209
vault: Fix lint warnings
bingen Mar 17, 2018
869ad3a
vault: Remove redundant variables
bingen Mar 17, 2018
c503098
vault: Remove constructor, refactor initialize
bingen Mar 17, 2018
dfb3ad7
templates: Remove EtherToken from tests
bingen Mar 16, 2018
d8dac5a
[wip] vault: Try to increase coverage
bingen Mar 18, 2018
42c954d
Merge branch 'beta_test_remove_ethertoken' of github.com:aragon/arago…
izqui Mar 18, 2018
File filter...
Filter file types
Jump to…
Jump to file or symbol
Failed to load files and symbols.
+0 −10
Diff settings

Always

Just for now

vault: Remove redundant variables

They are already in VaultBase
  • Loading branch information...
bingen committed Mar 17, 2018
commit 869ad3a2d8766780e5735dcece3e3e9fe15f9702
@@ -1,7 +1,6 @@
pragma solidity 0.4.18;

import "./VaultBase.sol"; // split made to avoid circular import
import "./IVaultConnector.sol";

import "./connectors/ERC20Connector.sol";
import "./connectors/ETHConnector.sol";
@@ -10,15 +9,6 @@ import "@aragon/os/contracts/lib/misc/Migrations.sol";


contract Vault is VaultBase {
address constant ETH = address(0);
uint32 constant ERC165 = 165;
uint32 constant NO_DETECTION = uint32(-1);

// connectors can define their own extra roles, challenge for discoverability
bytes32 constant REGISTER_TOKEN_STANDARD = keccak256("REGISTER_TOKEN_STANDARD");
bytes32 constant TRANSFER_ROLE = keccak256("TRANSFER_ROLE");
// TODO: Abstract over different APPROVAL and have just one role?

struct TokenStandard {
uint32 erc;
uint32 interfaceDetectionERC;
ProTip! Use n and p to navigate between commits in a pull request.
You can’t perform that action at this time.