Make StateMetadataDocument
public
#1085
Merged
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.
Description of change
We currently get warnings that
StateMetadataDocument::{into_iota_document,unpack}
are unused when runningcargo check --no-default-features
.This PR makes
StateMetadata
public to get rid of those warnings. Another approach would have been to feature-gate everything state metadata related behind theclient
feature. The reasoning for making it public is thatStateMetadata
can be used with clients other thaniota-client
and there's no inherent reason why it has to be tied to that specific client. We may also want to use it from the Wasm bindings without adding a dependency to iota-client in order to decode the state metadata of an Alias Output resolved by, say,iota.js
.Eq
was also derived forStateMetadataDocument
to satisfy a clippy lint.Links to any relevant issues
n/a
Type of change
Add an
x
to the boxes that are relevant to your changes.How the change has been tested
None.
Change checklist
Add an
x
to the boxes that are relevant to your changes.