-
Notifications
You must be signed in to change notification settings - Fork 8
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
V2.4.0. protocol initialization #838
Conversation
@@ -94,21 +94,23 @@ contract ProtocolInitializationHandlerFacet is IBosonProtocolInitializationHandl | |||
} | |||
} | |||
|
|||
ProtocolLib.ProtocolStatus storage status = protocolStatus(); | |||
// ProtocolLib.ProtocolStatus storage status = protocolStatus(); |
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.
remove comment
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.
Removed
/** | ||
* @notice Initializes the version 2.4.0. | ||
* | ||
* Initliaziation data is used to back-fill the royalty recipients for existing offers and sellers. |
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.
* Initliaziation data is used to back-fill the royalty recipients for existing offers and sellers. | |
* Initialization data is used to back-fill the royalty recipients for existing offers and sellers. |
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.
Corrected
* Supplied royalty percentage should match the current percentage set in seller's boson voucher contract. | ||
* If seller has multiple collections with different royalty precentages: | ||
* - the sellerId should be included in the group, corresponding to the minimal royalty percentage | ||
* - the _offerIds array should should be included in the group, corresponding to the royalty percentage that matches the royalty of the collection to which offer belongs |
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.
* - the _offerIds array should should be included in the group, corresponding to the royalty percentage that matches the royalty of the collection to which offer belongs | |
* - the _offerIds array should be included in the group, corresponding to the royalty percentage that matches the royalty of the collection to which offer belongs |
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.
Corrected
/** | ||
* @notice Method to initialize the protocol if it cannot be done in a single transaction. | ||
* | ||
* This shuld be used only if the amount of data is too large, and it cannot be done in a single `initialize` transaction. |
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.
* This shuld be used only if the amount of data is too large, and it cannot be done in a single `initialize` transaction. | |
* This should be used only if the amount of data is too large, and it cannot be done in a single `initialize` transaction. |
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.
Corrected
Due to recent changes, existing sellers are missing
royaltyRecipients
and offers are missingroyaltyInfo
.This data should be filled in during the initialization of the next version.
This PR adds
initV2_4_0
which is used to populate missing data.