Skip to content

Update NFT bridge docs#415

Merged
InoMurko merged 8 commits intodocs-in-monrepofrom
update-nft-bridge-docs
Oct 20, 2022
Merged

Update NFT bridge docs#415
InoMurko merged 8 commits intodocs-in-monrepofrom
update-nft-bridge-docs

Conversation

@boyuan-chen
Copy link
Copy Markdown
Contributor

📋 Add associated issues, tickets, docs URL here.

Overview

Describe what your Pull Request is about in a few sentences.

Rewrite NFT bridge docs

Changes

Describe your changes and implementation choices. More details make PRs easier to review.

  • Add requirements for NFT contracts
  • Move two NFT README into one

Testing

Describe how to test your new feature/bug fix and if possible, a step by step guide on how to demo this.

Comment thread boba_documentation/developer/nft-bridge.md Outdated
Comment thread boba_documentation/developer/nft-bridge.md
@souradeep-das
Copy link
Copy Markdown
Contributor

should we merge the complete 'More advanced options for Special NFTs' from here so all cases of metadata is covered?

Copy link
Copy Markdown
Contributor

@wsdt wsdt left a comment

Choose a reason for hiding this comment

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

looks good to me after adding Ino's feedback

@boyuan-chen
Copy link
Copy Markdown
Contributor Author

should we merge the complete 'More advanced options for Special NFTs' from here so all cases of metadata is covered?

It's added.

Comment thread boba_documentation/developer/nft-bridge.md Outdated
Co-authored-by: Ino Murko <ino.murko.github@protonmail.com>
@InoMurko InoMurko merged commit 2b86ea5 into docs-in-monrepo Oct 20, 2022
@InoMurko InoMurko deleted the update-nft-bridge-docs branch October 20, 2022 19:39
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.

4 participants