-
Notifications
You must be signed in to change notification settings - Fork 11
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
[EPIC] New fee model architecture #270
base: main
Are you sure you want to change the base?
Conversation
wip: change of plans was a mistakerino
`ManagedAllowList`
Co-authored-by: Manuel Montenegro <manuel@nucypher.com>
Extract fees logic from Coordinator
BqETH subscription (no encryptor fees yet)
Co-authored-by: David Núñez <david@nucypher.com>
Encryptor fees and slots management in subscription
General improvements for Subscription
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.
Looks like we will need some nucypher
updates to accommodate the new architecture. Filed nucypher/nucypher#3514.
contracts/contracts/coordination/subscription/BqETHSubscription.sol
Outdated
Show resolved
Hide resolved
return 42 * 10 ** 20; // TODO | ||
} | ||
|
||
/** | ||
* @notice Returns the base expiration duration | ||
* @return The base expiration duration | ||
*/ | ||
function baseExpiration() public pure returns (uint256) { | ||
return 52 weeks; // TODO |
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.
Do these "TODOs" still need to be revisited?
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 plan to remove Subscription contract when ManagedAllowList will be integrated with new Subscription
Co-authored-by: Derek Pierre <derek.pierre@gmail.com>
8e7bb18
to
3886874
Compare
Follow up PR: Removes BetaProgramInitiator, Global is Global, return backward compatibility
Type of PR:
Required reviews:
What this does:
Changes fee model, now it's separate contract that must be approved in coordinator.
Implements subscription for BqETH
Issues fixed/closed:
Why it's needed:
Notes for reviewers:
This PR is ready for tests and integration with nucypher repo, ManagedAllowList needs touch up in some of following PRs