Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Nfts attribute read interface #13349

Merged

Conversation

mrshiposha
Copy link
Contributor

@mrshiposha mrshiposha commented Feb 9, 2023

This PR contains changes to the frame_support::trait::tokens::nonfungible(s)_v2::Inspect trait:

  • attribute now doesn't take the namespace argument
  • Added custom_attribute function
  • Added system_attribute function
  • The typed-companions are also added

A runtime API is added for the NFTs pallet. The interface is the same as in the Inspect trait.

Some docs were modified.

Cumulus Companion: paritytech/cumulus#2217

@mrshiposha
Copy link
Contributor Author

@jsidorenko please take a look :)

@mrshiposha mrshiposha changed the title Nfts get attribute Nfts attribute read interface Feb 9, 2023
@jsidorenko jsidorenko added A0-please_review Pull request needs code review. B3-apinoteworthy C1-low PR touches the given topic and has a low impact on builders. D5-nicetohaveaudit ⚠️ PR contains trivial changes to logic that should be properly reviewed. labels Feb 9, 2023
@the-right-joyce the-right-joyce added B1-note_worthy Changes should be noted in the release notes T1-runtime This PR/Issue is related to the topic “runtime”. and removed B3-apinoteworthy labels Feb 13, 2023
Copy link
Contributor

@jsidorenko jsidorenko left a comment

Choose a reason for hiding this comment

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

👍

@mrshiposha
Copy link
Contributor Author

I'm sorry for the force push :(
I forgot about the rule. But I will remember it in the future...

@jsidorenko
Copy link
Contributor

bot merge

@paritytech-processbot
Copy link

Error: Github API says paritytech/cumulus#2217 is not mergeable

@jsidorenko
Copy link
Contributor

bot merge

@paritytech-processbot paritytech-processbot bot merged commit 91b4739 into paritytech:master Feb 23, 2023
ark0f pushed a commit to gear-tech/substrate that referenced this pull request Feb 27, 2023
* feat: add custom and system attributes to Inspect

* feat: add nfts runtime api

* fix: pass std feature to runtime api

* fix: api copyright

Co-authored-by: joe petrowski <[email protected]>

---------

Co-authored-by: joe petrowski <[email protected]>
ukint-vs pushed a commit to gear-tech/substrate that referenced this pull request Apr 10, 2023
* feat: add custom and system attributes to Inspect

* feat: add nfts runtime api

* fix: pass std feature to runtime api

* fix: api copyright

Co-authored-by: joe petrowski <[email protected]>

---------

Co-authored-by: joe petrowski <[email protected]>
@redzsina redzsina added D1-audited 👍 PR contains changes to fund-managing logic that has been properly reviewed and externally audited and removed D5-nicetohaveaudit ⚠️ PR contains trivial changes to logic that should be properly reviewed. labels Apr 14, 2023
nathanwhit pushed a commit to nathanwhit/substrate that referenced this pull request Jul 19, 2023
* feat: add custom and system attributes to Inspect

* feat: add nfts runtime api

* fix: pass std feature to runtime api

* fix: api copyright

Co-authored-by: joe petrowski <[email protected]>

---------

Co-authored-by: joe petrowski <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review. B1-note_worthy Changes should be noted in the release notes C1-low PR touches the given topic and has a low impact on builders. D1-audited 👍 PR contains changes to fund-managing logic that has been properly reviewed and externally audited T1-runtime This PR/Issue is related to the topic “runtime”.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants