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

feature request: Storage query of uniques classes by account #9777

Closed
Yuripetusko opened this issue Sep 14, 2021 · 2 comments · Fixed by #9940
Closed

feature request: Storage query of uniques classes by account #9777

Yuripetusko opened this issue Sep 14, 2021 · 2 comments · Fixed by #9940
Assignees
Labels
J2-unconfirmed Issue might be valid, but it’s not yet known.

Comments

@Yuripetusko
Copy link

We are working on UI for minting uniques assets and noticed that there's no way to query Classes by account, only assets. I can query all asset keys first by account, then query all classes from there and filter by those where issuer === accountId, but this won't work for Classes that have no assets (empty classes)

So I am wondering if it possible to add such storage query to substrate.

The reason why it's a problem. Imagine a UI/UX where you first create a collection (class) and then mint NFTs (assets) into this collection.

Imagine if user refreshes a page between signing class create tx and asset mint tx, so now there's no way to display all his previously created empty classes to select the one where to mint first asset into

@github-actions github-actions bot added the J2-unconfirmed Issue might be valid, but it’s not yet known. label Sep 14, 2021
@hamidra hamidra self-assigned this Sep 30, 2021
@gavofyork
Copy link
Member

Is it not possible to enumerate all Classes and filter those created by the user?

@Yuripetusko
Copy link
Author

Yuripetusko commented Oct 6, 2021

Is it not possible to enumerate all Classes and filter those created by the user?

I am worried that query will be too heavy / slow once people start minting uniques actively and there are thousands of classes

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
J2-unconfirmed Issue might be valid, but it’s not yet known.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants