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

add inspect trait for asset roles #11738

Conversation

girazoki
Copy link
Contributor

It adds a new fungibles trait to inspect roles for assetIds. In particular it adds getters for owner, issuer, freezer, admin

@kianenigma kianenigma changed the title Girazoki add inspect trait for asset roles add inspect trait for asset roles Jun 30, 2022
@stale
Copy link

stale bot commented Jul 30, 2022

Hey, is anyone still working on this? Due to the inactivity this issue has been automatically marked as stale. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the A5-stale Pull request did not receive any updates in a long time. No review needed at this stage. Close it. label Jul 30, 2022
Copy link
Contributor

@KiChjang KiChjang left a comment

Choose a reason for hiding this comment

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

This looks fair enough to me.

@stale stale bot removed the A5-stale Pull request did not receive any updates in a long time. No review needed at this stage. Close it. label Jul 30, 2022
@KiChjang KiChjang added A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit labels Jul 30, 2022
@stale
Copy link

stale bot commented Aug 31, 2022

Hey, is anyone still working on this? Due to the inactivity this issue has been automatically marked as stale. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the A5-stale Pull request did not receive any updates in a long time. No review needed at this stage. Close it. label Aug 31, 2022
@stale stale bot removed the A5-stale Pull request did not receive any updates in a long time. No review needed at this stage. Close it. label Sep 12, 2022
@shawntabrizi
Copy link
Member

bot merge

@paritytech-processbot
Copy link

This PR cannot be merged at the moment due to: 5 of 12 required status checks are expected.

processbot expects that the problem will be solved automatically later and so the auto-merge process will be started. You can simply wait for now.

@paritytech-processbot
Copy link

Merge cancelled due to error. Error: Head SHA changed from b412a53 to dc30523

@shawntabrizi
Copy link
Member

bot merge

@paritytech-processbot
Copy link

Waiting for commit status.

@shawntabrizi shawntabrizi added B3-apinoteworthy and removed B0-silent Changes should not be mentioned in any release notes B3-apinoteworthy labels Sep 13, 2022
@paritytech-processbot paritytech-processbot bot merged commit 5daa20c into paritytech:master Sep 13, 2022
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. C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants