Apply my suggestions to the PR #5
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ligi
approved these changes
Jul 18, 2022
|
thanks! |
ligi
pushed a commit
that referenced
this pull request
Feb 10, 2023
* Propose Multi-Resource Token standard RMRK team has developed a next step in NFTs where one NFT can be tied to multiple resources. * Address issues reported by EIP repository's CI * Fix styling discrepancies in section titles * Address CI's issues in the preabmle * Add discussion URL * Fix discussion URL formatting * Add the EIP number * Fix getResourceMeta specification * Apply changes based on PR comments * Replace "primitive" with a clearer explanation Co-authored-by: Pandapip1 <45835846+Pandapip1@users.noreply.github.com> * Apply changes based on ECH call #5 Applied the changes based on feedback received on the EIP Editing Office Hour Meeting #5. This commit includes changes to the proposal as well as adds an exaple implementation to assets/ directory. * Address the stylistic issues reported by the CI * Fix a reference to EIP-712 * Minor typo fixes * Updates `getResourceMetadata` description Co-authored-by: Sam Wilson <57262657+SamWilsn@users.noreply.github.com> * Rename MultiResource -> MultiAsset and relicense The proposal was renamed to Context-Dependent Multi-Asset Tokens to better illustrate its function. Another example was added to represent the possible IoT usecase and the explanation on the naming decision was added to the rationale. The examples were relicensed to CC0, to conform to the requirements of EIP repository. * Fix license identifier & Specification contract title * Fix indexes of Rationale questions * Minor polishing of the rationale * Update references to assets to use an istead of a in Rationale * Final polishes Co-authored-by: Pandapip1 <45835846+Pandapip1@users.noreply.github.com> Co-authored-by: Steven Pineda <steven2308@outlook.com> Co-authored-by: Sam Wilson <57262657+SamWilsn@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.