Skip to content
This repository has been archived by the owner on Apr 30, 2024. It is now read-only.

Add Licensing-Royalty integration and new LAP royalty policy #99

Merged
merged 22 commits into from
Feb 17, 2024

Conversation

jdubpark
Copy link
Contributor

@jdubpark jdubpark commented Feb 16, 2024

  • Adds internal hooks for royalty and license integration
  • Adds the new LAP royalty policy and Ancestors vaults contracts and unit tests

Rebase + test fixing from #94 (replace and closes #94)

@Ramarti @Spablob Please explain the new royalty and licensing system here.

@jdubpark jdubpark marked this pull request as ready for review February 16, 2024 23:31
@jdubpark jdubpark requested a review from a team February 16, 2024 23:33
Spablob and others added 14 commits February 16, 2024 18:58
add license integration hooks to royalty module

Add royalty policy LAP and ancestors vault

comment out required files for integration

adjustment in the errors.sol file

add array utils lib

adjust git.ignore file

remove parentIpIds from onLicenseMinting

minor spelling fix

fixes related with removing parentIpIds from onLicenseMinting

add natspec on onLinkToParents

add license data and external data inputs

adjust interfaces

adjust error.sol file

adjust ancestorsVault contract and unit tests

adjust royalty policy LAP contract and unit tests

test fixes
fixing tests

delete LSClaimer and RoyaltyPolicyLS

add payRoyaltyOnBehalf unit tests

fixed some tests
@jdubpark jdubpark force-pushed the limited-tree-royalty-policy branch from fcb1142 to 8163d37 Compare February 17, 2024 01:33
Copy link
Contributor

@LeoHChen LeoHChen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

a few minor comments

@LeoHChen LeoHChen merged commit 8bd18c0 into storyprotocol:main Feb 17, 2024
1 check passed
kingster-will referenced this pull request in kingster-will/protocol-core-v1-dev Mar 19, 2024
…otocol#99)

* Add Licensing-Royalty integration and new LAP royalty policy

* add license integration hooks to royalty module

* Add royalty policy LAP and ancestors vault

* add natspec on onLinkToParents

* add license data and external data inputs

* adjust interfaces

* adjust ancestorsVault contract and unit tests

* adjust royalty policy LAP contract and unit tests

* feat: License + Royalty integration storyprotocol#94 off rebase

* test: fixing tests and flows

* fix: royalty policy onLicenseMinting pulled out to correct position

* test: Integration fix for SingleNftCollection test

---------

Co-authored-by: Spablob <[email protected]>
Co-authored-by: Raul <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants