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

feat(docs): add NFT cookbook #958

Merged
merged 4 commits into from
Oct 25, 2024
Merged

feat(docs): add NFT cookbook #958

merged 4 commits into from
Oct 25, 2024

Conversation

a-bahdanau
Copy link
Contributor

@a-bahdanau a-bahdanau commented Oct 14, 2024 •
edited by novusnota
Loading

Issue

Closes #957 and Closes tact-lang/tact-docs#322

Checklist

  • I have updated CHANGELOG.md
  • I have run the linter, formatter and spellchecker
  • I did not do unrelated and/or undiscussed refactorings

@a-bahdanau a-bahdanau requested a review from a team as a code owner October 14, 2024 09:14
@anton-trunov anton-trunov added this to the v1.6.0 milestone Oct 23, 2024
@Gusarich
Copy link
Member

Didn't we already review this one in tact-docs repo? Should we review it again, what was changed?

novusnota
novusnota requested changes Oct 25, 2024
View reviewed changes
Copy link
Member

@novusnota novusnota left a comment

Choose a reason for hiding this comment

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

Yeah, content-wise this PR was already approved by @Gusarich, so nothing changed there. I just want to clean-up the formatting for the new docs and we can merge this

docs/src/content/docs/cookbook/nfts.mdx Outdated Show resolved Hide resolved
docs/src/content/docs/cookbook/nfts.mdx Outdated Show resolved Hide resolved
docs/src/content/docs/cookbook/nfts.mdx Outdated Show resolved Hide resolved
@novusnota novusnota mentioned this pull request Oct 25, 2024
Closed
@novusnota novusnota merged commit 1adf5d5 into tact-lang:main Oct 25, 2024
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@novusnota novusnota novusnota approved these changes

@Gusarich Gusarich Awaiting requested review from Gusarich

Assignees

@novusnota novusnota

Labels
None yet
Projects
None yet
Milestone
v1.6.0
Development

Successfully merging this pull request may close these issues.

Working with NFTs in Tact Working with NFTs in Tact
4 participants
@a-bahdanau @Gusarich @novusnota @anton-trunov