Skip to content

Refactor existing native composite values to use CompositeValue #650

Closed

Description

Context

Currently native composite values (e.g. AuthAccount, AuthAccount.Contracts, PublicAccount, etc.) are defined through separate Go types implementing the Value interface (e.g. AuthAccountValue).

Refactor those existing value type to use CompositeValue instead.

Pros:

  • No need to declare new Value type, reduce boilerplate
  • No special conversion functions in internal encoding/decoding and import/export

Definition of Done

Non-exhaustive list:

  • AuthAccountValue
  • PublicAccountValue
  • AuthAccountContractsValue

Not good candidates:

  • PathValue
  • LinkValue
  • StringValue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Assignees

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions