CodeChain-io / CodeChain-io/codechain-sdk-go

Use custom JSON marshal instead of object duplication

Open
#54 2 comments 0 reactions 0 assignees View on GitHub
Dominant language
Go
Stars
5
Forks
4
PR merge metrics
No merged PRs in 30d

Description

Currently, some of the Transaction-related objects have two forms, which implement normal type and for marshaling.

https://github.com/CodeChain-io/codechain-sdk-go/blob/6ad3e76c4dfba22b0867764b8c67500dce7518d2/core/transaction/signedTransaction.go#L11-L17
https://github.com/CodeChain-io/codechain-sdk-go/blob/6ad3e76c4dfba22b0867764b8c67500dce7518d2/core/transaction/signedTransaction.go#L19-L25

This implementation is from [codechain-sdk-js](https://github.com/CodeChain-io/codechain-sdk-js).
But since golang can implement custom marshaling function ([link1](http://choly.ca/post/go-json-marshalling/
)) ([link2](https://play.golang.org/p/CTmJhTiGAM)), I think duplication of the objects is unnecessary.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.