You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Also we would like to ask you to make some flag to disable token_registry_metadata.logo field in asset_info GET&POST endpoints (https://api.koios.rest/#post-/asset_info), as it can be disabled only through full filtering of token_registry_metadatajsonb object.
In our lightwallet, updating asset data does not always require a logo, but it does require decimals for example. So requests are sometimes 1-2-5 megabytes.
Quick access link (we just made a separate RPC *crazy*):
@ray-robot - hope you dont mind converting that comment to seperate issue here.
Unlike the utxo endpoints in previous issue where we were adding _extended to avoid joins (performance impact), the requirement here is more around verticle filtering for contents within endpoint with details from same table.
Hence, I'd want to be sure to understand that (since consumption is programatic and not manual), wouldnt something like below fit the purpose?
In general, this (exclusion) would then kinda extend to other fields across different endpoints wherein you might want to exclude transaction metadata details or datum byte which will be similarly large footprints unless filtered. Let us know your views around the same.
Also we would like to ask you to make some flag to disable
token_registry_metadata.logo
field inasset_info
GET&POST endpoints (https://api.koios.rest/#post-/asset_info), as it can be disabled only through full filtering oftoken_registry_metadata
jsonb
object.In our lightwallet, updating asset data does not always require a
logo
, but it does requiredecimals
for example. So requests are sometimes 1-2-5 megabytes.Quick access link (we just made a separate RPC
*crazy*
):https://github.com/ray-network/raygraph-output/blob/d21d2a6a9a269485251f990e74254c95ebc812c4/koios-lite/rpc-extra/assets_info_bulk_no_logo.sql#L55
Originally posted by @ray-robot in #186 (comment)
The text was updated successfully, but these errors were encountered: