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 #94

Closed
wants to merge 21 commits into from

Conversation

Spablob
Copy link
Contributor

@Spablob Spablob commented Feb 15, 2024

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

@Spablob Spablob marked this pull request as ready for review February 16, 2024 13:51
@Spablob Spablob closed this Feb 17, 2024
LeoHChen pushed a commit that referenced this pull request Feb 17, 2024
* 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 #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]>
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.

2 participants