Skip to content

Remove default types from AminoTypes #1079

Closed
@webmaster128

Description

Right now, AminoTypes works by having a set of default types and additions.

However, that conflicts with the idea of having independent modules and very different client configurations for each chain. It also makes testing hard.

  • Let The AminoTypes class take a AminoConverters (same as Record<string, AminoConverter | "not_supported_by_chain">) argument in the constructor
  • Move function createDefaultTypes(prefix: string): AminoConverters to the SigningStargateClient
  • Export all create*AminoConverters in @cosmjs/stargate
  • Use only auth, bank and wasm Amino converter types in the AminoTypes instance of the SigningCosmWasmClient
  • Split test from packages/stargate/src/aminotypes.spec.ts and move them to src/modules/*/aminomessages.spec.ts
  • Move AminoConverter/AminoConverters into packages/stargate/src/aminotypes.ts
  • CHANGELOG

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions