-
Notifications
You must be signed in to change notification settings - Fork 7
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
Fix NFT contract templates #83
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #83 +/- ##
=======================================
Coverage 86.19% 86.19%
=======================================
Files 18 18
Lines 884 884
=======================================
Hits 762 762
Misses 122 122 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you have tests for these new transactions?
I just wired up the vault & collection inspection scripts to tests. The storage iteration script to gather token info wasn't covered as I need to get some other changes in for the weekend. It's not mutating - purely informational - but could afford to add coverage in the future. |
Closes: #82
Description
Updates the NFT template across networks (redundantly included due to the need for static import declarations). These updates are then reflected in the Cadence JSON arguments as code chunks reflective of the contract changes.
Also adds queries to determine if an account is configured to receive a specified vault or NFT.
For contributor use:
main
branchFiles changed
in the Github PR explorer