Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add total_suppy field on account assets and address assets. #119

Closed
HT-Moh opened this issue Oct 24, 2022 · 3 comments
Closed

Add total_suppy field on account assets and address assets. #119

HT-Moh opened this issue Oct 24, 2022 · 3 comments
Assignees
Milestone

Comments

@HT-Moh
Copy link

HT-Moh commented Oct 24, 2022

At the moment the only way to filter assets policies (FT and NFT) is by using the total_supply, if we want to filter account assets we need to do a call to asset_info for each asset, so for a wallet with 100 assets_info we need 100 calls, this lot of network requests.
is there a possibility to add total_supply to account_assets and address_assets?
The CIP-0067 gives the possibility to classify the asset from the name but it's not adopted yet ;-)

@HT-Moh HT-Moh changed the title Add total_suppy field on account assets. Add total_suppy field on account assets and address assets. Oct 24, 2022
@rdlrt rdlrt transferred this issue from cardano-community/guild-operators Oct 26, 2022
@rdlrt rdlrt moved this to Next Up in Koios Project Oct 26, 2022
@rdlrt rdlrt added this to the Koios vNext milestone Oct 26, 2022
@rdlrt rdlrt moved this from Next Up to Todo in Koios Project Nov 2, 2022
@rdlrt rdlrt moved this from Todo to Next Up in Koios Project Nov 3, 2022
@rdlrt
Copy link
Contributor

rdlrt commented Nov 3, 2022

TBD - consideration to be put in asset object consistency, will have a dependency on cache table - to be deferred to release post koios-1.0.9

@Scitz0
Copy link
Contributor

Scitz0 commented Nov 17, 2022

Asset cache table should include the following fields:
asset_id, total_supply, first_mint_tx_id, first_mint_keys, last_mint_tx_id, last_mint_keys, decimals

@Scitz0
Copy link
Contributor

Scitz0 commented Nov 17, 2022

Superseded by #124

@Scitz0 Scitz0 closed this as completed Nov 17, 2022
Repository owner moved this from Next Up to Testing in Koios Project Nov 17, 2022
@rdlrt rdlrt moved this from Testing to Completed in Koios Project Dec 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Completed
Development

No branches or pull requests

3 participants