Allow Contributors management by core contirbutors #17

2018-04-08T11:00:34Z

bumi commented

2018-04-08 11:00:34 +00:00

(Migrated from github.com)

So far we only allowed calls to the Contributors contract from the
Operator. With the new registry concept we can call functions again
directly on the Contributors contract (without the need to call it through
the operator).
This changes the authentication for the contributor management functions
to allow either core contributors or the operator to call them.

In the future I envision a bit more flexible and configurable
authentication concept that can more easily evolve over time.

So far we only allowed calls to the Contributors contract from the Operator. With the new registry concept we can call functions again directly on the Contributors contract (without the need to call it through the operator). This changes the authentication for the contributor management functions to allow either core contributors or the operator to call them. In the future I envision a bit more flexible and configurable authentication concept that can more easily evolve over time.

bumi commented

2018-04-08 11:05:37 +00:00

(Migrated from github.com)

This basically also deprecates these functions: https://github.com/67P/truffle-kredits/blob/master/contracts/Operator.sol#L56-L70
Which I will remove in favor of calling the contract directly.

As mentioned the registry makes it trivial to discover the contract address. The contracts are still kept small and specific but should also have some autonomy.
The upgradable proxy concept allows us to make some modifications to the contract. (hopefully)

This basically also deprecates these functions: https://github.com/67P/truffle-kredits/blob/master/contracts/Operator.sol#L56-L70 Which I will remove in favor of calling the contract directly. As mentioned the registry makes it trivial to discover the contract address. The contracts are still kept small and specific but should also have some autonomy. The upgradable proxy concept allows us to make some modifications to the contract. (hopefully)

fsmanuel (Migrated from github.com) approved these changes 2018-04-08 12:12:21 +00:00

fsmanuel (Migrated from github.com) left a comment

Much better!

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: kredits/contracts#17