Refactor/remove serde prefixes#9

Merged

Alrighttt merged 30 commits intorelease_candidate_1from

refactor/remove-serde-prefixes

Dec 5, 2024

Commits on Nov 15, 2024

refactor Hash256 serde to account for Sia Go SiaFoundation/core/pull/224
Alrighttt
committed
remove now irrelevant dev comment
Alrighttt
committed
remove h: prefix from all unit tests
Alrighttt
committed
refactor Address serde to remove addr: prefix
Alrighttt
committed
remove addr: prefix from all Address instances in unit tests
Alrighttt
committed
fix merkle_proof field in StateElement serde - walletd is capable of providing a JSON that does not include this field, includes it as `null` or includes it as empty array
Alrighttt
committed
add more explitcit unit test for StateElement serde
Alrighttt
committed
update serde test to account for prefix removals and state element changes
Alrighttt
committed
rename symbol BlockID -> BlockId
Alrighttt
committed
derive From<BlockId> for Hash256 and vice versa instead of manually implementings
Alrighttt
committed
remove unused import
Alrighttt
committed
remove bid: prefix from BlockId serde
Alrighttt
committed
update SiacoinElement unit test to account for type changes
Alrighttt
committed
refactor Signature serde to remove sig: prefix
Alrighttt
committed
use Signature::try_from instead of now removed Signature::from_bytes
Alrighttt
committed
update serde tests v2ContractResolution event expected case JSON
Alrighttt
committed
manually impl Serialize Deserialize for Hash256 for better error messages
Alrighttt
committed
remove prefix from varoius unit tests
Alrighttt
committed
fix V2FileContractElement field name serde
Alrighttt
committed
ignore several tests that rely on hardcoded walletd server
Alrighttt
committed
fix expected case in various events unit tests
Alrighttt
committed
cargo clippy
Alrighttt
committed

Commits on Nov 18, 2024

add explicit ArbitraryData type and implement serde
Alrighttt
committed
add ArbitraryData roundtrip serde unit test
Alrighttt
committed
derive From, Into for ArbitraryData newtype
Alrighttt
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Refactor/remove serde prefixes#9

Refactor/remove serde prefixes#9
Alrighttt merged 30 commits intorelease_candidate_1from
refactor/remove-serde-prefixes

Commits on Nov 15, 2024

refactor Hash256 serde to account for Sia Go SiaFoundation/core/pull/224

remove now irrelevant dev comment

remove h: prefix from all unit tests

refactor Address serde to remove addr: prefix

remove addr: prefix from all Address instances in unit tests

fix merkle_proof field in StateElement serde - walletd is capable of providing a JSON that does not include this field, includes it as `null` or includes it as empty array

add more explitcit unit test for StateElement serde

update serde test to account for prefix removals and state element changes

rename symbol BlockID -> BlockId

derive From<BlockId> for Hash256 and vice versa instead of manually implementings

remove unused import

remove bid: prefix from BlockId serde

update SiacoinElement unit test to account for type changes

refactor Signature serde to remove sig: prefix

use Signature::try_from instead of now removed Signature::from_bytes

update serde tests v2ContractResolution event expected case JSON

manually impl Serialize Deserialize for Hash256 for better error messages

remove prefix from varoius unit tests

fix V2FileContractElement field name serde

ignore several tests that rely on hardcoded walletd server

fix expected case in various events unit tests

cargo clippy

Commits on Nov 18, 2024

add explicit ArbitraryData type and implement serde

add ArbitraryData roundtrip serde unit test

derive From, Into for ArbitraryData newtype

Commits on Nov 19, 2024

fix ConsensusTipstateResponse serde

allow unknown fields in V2Transaction serde because walletd will sometimes include "id" field

fix trailing ? on endpoint Urls that have query params but none are provided

increase native client timeout from 10 seconds to 30

add ApiClientHelpers::get_unconfirmed_transaction and dev comments related to ApiClientHelpers::get_transaction