Ensure only data blobs are present in SDK/System API calls #2368
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Motivation
We don't want
BlobId
exposed to the SDK. Users will only be aware ofData
blobs, the other types will be only internally used.Proposal
Only expose
Data
blobs to the SDK, and assume when dealing with blobs that it's aData
blob. Because of that we don't need to useBlobId
everywhere, and can just use the hash of theData
blob in most places.Test Plan
CI + also retested the NFT UI