metaplex-foundation / digital-asset-validator-plugin

The NFT Geyser plugin that powers metaplex APIs
GNU Affero General Public License v3.0
33 stars 20 forks source link

Recommend addition to the `getAssetProof` response #55

Closed nickfrosty closed 1 year ago

nickfrosty commented 1 year ago

Problem

When building a transfer instruction for compressed NFTs, developers and sdks are required to get a valid proof via the getAssetProof method. But also the asset's data_hash and creator_hash from the getAsset method too.

The end result is that nearly every time a developer needs to build the transfer instruction, they have to make 2 RPC calls to get all the data that is needed to completely build the transfer instruction (or they have to already have some values cached elsewhere).

Solution

When fetching data via the getAssetProof method, I recommend adding the asset's current data_hash and creator_hash within the response.

Maybe even include the entire compression field data from the getAsset method?

nickfrosty commented 1 year ago

Moved to: https://github.com/metaplex-foundation/digital-asset-rpc-infrastructure/issues/54