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

feat(licensing): add maxRevenueShare parameter to align with core protocol changes #145

Merged
merged 1 commit into from
Dec 16, 2024

Conversation

sebsadface
Copy link
Member

Description

This PR adds the maxRevenueShare parameter to functions and tests related to derivative registration and license token minting, aligning with changes introduced in Core Protocol PR#364.

The maxRevenueShare parameter enables users to define the maximum royalty percentage they are willing to accept when minting license tokens or registering derivatives to parent IPs. This enhancement improves user control and flexibility in licensing agreements.

@sebsadface sebsadface marked this pull request as ready for review December 16, 2024 08:23
@kingster-will
Copy link

Do we provide an interface without the new parameters, so that application integrated with 1.2 api won't need to be changed?

@sebsadface
Copy link
Member Author

sebsadface commented Dec 16, 2024

Do we provide an interface without the new parameters, so that application integrated with 1.2 api won't need to be changed?

The plan is to add backward compatibility all together in a separate PR, we have an issue tracking this: #131

@sebsadface sebsadface merged commit 52f2241 into storyprotocol:main Dec 16, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants