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

Merge IPRecordRegistry and IPAccountRegistry into Unified IPAssetRegistry with Permissionless Registration #46

Merged
merged 2 commits into from
Jan 31, 2024

Conversation

kingster-will
Copy link
Contributor

@kingster-will kingster-will commented Jan 30, 2024

This PR introduces changes to the structure of our IP management. We have merged the IPRecordRegistry and IPAccountRegistry into a unified IPAssetRegistry. This simplifies the management of IP assets and provides a single point of interaction for these assets.

Additionally, we have updated the registration process to allow for permissionless registration of IP. This change enhances the openness and accessibility of our platform, allowing any user to register their IP without needing specific permissions or roles.

Note

Please note that the access control for setting metadata into MetadataProvider has not yet been implemented in this PR. This will be addressed in a future PR.

@kingster-will kingster-will changed the title WIP: Merge IPRecordRegistry and IPAccountRegistry Merge IPRecordRegistry and IPAccountRegistry Jan 31, 2024
@kingster-will kingster-will changed the title Merge IPRecordRegistry and IPAccountRegistry Merge IPRecordRegistry and IPAccountRegistry into Unified IPAssetRegistry with Permissionless Registration Jan 31, 2024
@kingster-will kingster-will self-assigned this Jan 31, 2024
@kingster-will kingster-will marked this pull request as ready for review January 31, 2024 03:37
Copy link
Contributor

@leeren leeren left a comment

Choose a reason for hiding this comment

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

LGTM

@kingster-will kingster-will merged commit b0aee07 into storyprotocol:main Jan 31, 2024
1 check passed
jdubpark pushed a commit that referenced this pull request Jan 31, 2024
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