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

Add Cross-Chain IP Registration Functionality #55

Merged
merged 4 commits into from
Apr 14, 2024

Conversation

kingster-will
Copy link

This PR introduces a new function to the IPAssetRegistry contract that allows for the registration of IP NFTs from different chain.

The new function, registerCrossChain, takes in three parameters: chainId, tokenContract, and tokenId. The chainId represents the identifier of the blockchain network where the IP NFT resides.

This function first checks if the chainId is the same as the current network's chainId. If they are the same, it throws an error as this function is specifically designed for cross-chain registrations.

@jdubpark
Copy link

Might require upgradability as cross-chain requirements will evolve and might impact IPAssetRegistry.
cc @Ramarti @LeoHChen

@LeoHChen
Copy link

Might require upgradability as cross-chain requirements will evolve and might impact IPAssetRegistry. cc @Ramarti @LeoHChen

valid concern, that's why I don't feel we shall set the IPAssetRegistry non-upgradable right now.

@LeoHChen
Copy link

The decision is to support cross chain mint in register with chain id added for now.

@kingster-will kingster-will force-pushed the v1/chainid-registration branch from d12f035 to 67b3f0c Compare April 14, 2024 02:35
Copy link

@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.

LGMT, please squash all commits when merge.

@storyprotocol storyprotocol deleted a comment from jdubpark Apr 14, 2024
@storyprotocol storyprotocol deleted a comment from LeoHChen Apr 14, 2024
@LeoHChen LeoHChen merged commit d9e60b3 into storyprotocol:main Apr 14, 2024
3 checks passed
jdubpark pushed a commit to jdubpark/sp-protocol-core that referenced this pull request Apr 14, 2024
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.

3 participants