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

Update NFTCatalog Cadence code for Cadence 1.0 #147

Open
sisyphusSmiling opened this issue Sep 29, 2023 · 3 comments
Open

Update NFTCatalog Cadence code for Cadence 1.0 #147

sisyphusSmiling opened this issue Sep 29, 2023 · 3 comments

Comments

@sisyphusSmiling
Copy link
Collaborator

Description

These contracts will need to be updated in preparation for Stable Cadence.

Context

While not directly used in NFTStorefront contracts, it looks like integration is useful and demonstrated in example transactions. Not currently a blocker, but flagging as I came across the contracts in onflow/nft-storefront#89
@sisyphusSmiling sisyphusSmiling changed the title Update NFTCatalog & NFTCatalog Admin for Cadence 1.0 Update NFTCatalog & NFTCatalogAdmin for Cadence 1.0 Sep 29, 2023
@sisyphusSmiling sisyphusSmiling changed the title Update NFTCatalog & NFTCatalogAdmin for Cadence 1.0 Update NFTCatalog Cadence code for Cadence 1.0 Sep 29, 2023
@sisyphusSmiling sisyphusSmiling mentioned this issue Sep 29, 2023
Merged
5 tasks
@austinkline
Copy link
Collaborator

Has this been started yet? Some of our contracts at flowty rely on the NFTCatalog

@bshahid331
Copy link
Contributor

Has this been started yet? Some of our contracts at flowty rely on the NFTCatalog

Yea it’s being worked on. There is a feature branch for this pushed up

@bshahid331
Copy link
Contributor

This is the branch: https://github.com/onflow/nft-catalog/tree/feature/cadence-1.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@bshahid331 @austinkline @sisyphusSmiling