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.

Sign up for GitHub

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

Jump to bottom

Add CrossVM views #242

Open
sisyphusSmiling opened this issue Feb 6, 2025 · 0 comments · May be fixed by #243
Open

Add CrossVM views #242

sisyphusSmiling opened this issue Feb 6, 2025 · 0 comments · May be fixed by #243
Assignees
@sisyphusSmiling
Labels
Feature Feedback

Comments

@sisyphusSmiling
Copy link
Collaborator

Related: onflow/flips#318

Description

Add the two views proposed in FLIP 318 - EVMPointer and EVMBytesMetadata.

Since adding these views to the standard MetadataViews would result in a circular dependency (EVM > MetadataViews > FlowToken > EVM), these views will need to be added to an alternate contract. While unfortunate and might normally result in reduced adoption, these views are ultimately only used by the bridge when moving NFTs between Cadence & EVM. Since the implementation of these views are a hard requirement for metadata to be passed into EVM, they will be leveraged in any documentation, guides and examples demonstrating dual VM deployment and can be assumed developers would see their documented use in the process of writing their contracts.
@sisyphusSmiling sisyphusSmiling self-assigned this Feb 6, 2025
@sisyphusSmiling sisyphusSmiling linked a pull request Feb 6, 2025 that will close this issue
Draft
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sisyphusSmiling sisyphusSmiling

Labels
Feature Feedback
Projects
🌊 Flow 4D
Status: No status
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add CrossVM views onflow/flow-nft
1 participant
@sisyphusSmiling