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
The SubmitClaims struct currently has a field named metadata. This name might be misleading as it conveys the idea of a data about data, which is not the quite the case for this action.
SubmitClaims{/// The serialized metadata intended for attachment./// This metadata should adhere to the format specified in the `format` field.metadata:Binary,/// RDF dataset serialization format for the metadata./// If not provided, the default format is [N-Quads](https://www.w3.org/TR/n-quads/) format.format:Option<RdfDatasetFormat>,},
Proposal
For clarity and precision, I propose to rename the metadata field to claims as a claim refers to a piece of information about an entity (such as a user, a service) that can be provided by one party and is potentially verifiable.
Impact
positive: this change will make the code more readable and self-explanatory
negative: this change introduces a breaking-change in the contract
neutral: none
The text was updated successfully, but these errors were encountered:
ccamel
changed the title
♻️ Rename "metadata" field in SubmitClaims struct to "claim"
♻️ Rename "metadata" field in SubmitClaims struct to "claims"
Jun 14, 2024
Proposal
The
SubmitClaims
struct currently has a field namedmetadata
. This name might be misleading as it conveys the idea of a data about data, which is not the quite the case for this action.Proposal
For clarity and precision, I propose to rename the
metadata
field toclaims
as a claim refers to a piece of information about an entity (such as a user, a service) that can be provided by one party and is potentially verifiable.Impact
The text was updated successfully, but these errors were encountered: