-
Notifications
You must be signed in to change notification settings - Fork 3
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
chore: deployment scripts #75
Conversation
kupermind
commented
Jan 10, 2025
- Deployment scripts.
if (_subscriptionNFT == address(0)) { | ||
revert ZeroAddress(); | ||
} | ||
|
||
if (_subscriptionTokenId == 0) { | ||
revert ZeroValue(); | ||
} | ||
|
||
subscriptionNFT = _subscriptionNFT; | ||
subscriptionTokenId = _subscriptionTokenId; |
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.
Subscription is set separately as it needs to be tied to this contract address
// Temporary owner address | ||
address public owner; |
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.
Temporary ownership is dropped when the subscription is set
/// @dev Process mech payment. | ||
/// @param mech Mech address. | ||
/// @return mechPayment Mech payment. | ||
function _processPayment(address mech) internal virtual override returns (uint256 mechPayment, uint256) { |
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 far as the understanding goes, all the fees are already accounted for when the subscription is paid for and mech gets its cleared cut.