Skip to content

Commit

Permalink
impl State for AuthorityState, so we can handle SuiError from Authori…
Browse files Browse the repository at this point in the history
…tyState as StateReadError. This will help us later when handling QuorumDriverErrror and SuiErrors that may arise from other situations

plug in the new State trait for coin_api, fix some typing issues

onboard indexer_api to state trait object

transaction_builder_api uses Arc<dyn State>

onboard transaction_execution_api to StateReadError

read_api use Arc<dyn State>

that's the last one, move_utils

clean up coin_api unit tests and error handling

another runthrough to check AuthorityState usage

map StateReadErrors to -32001 error code

fix unit test

Ok, this nested thing might turn out to be annoying in the end, but we'll see. Maybe we can just map to anyhow instead

add some docstrings

keep the breaking changes minimal
  • Loading branch information
wlmyng committed Jul 19, 2023
1 parent fc8ff4b commit ecb799c
Show file tree
Hide file tree
Showing 14 changed files with 755 additions and 197 deletions.
1 change: 1 addition & 0 deletions Cargo.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 1 addition & 0 deletions crates/sui-json-rpc/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@ publish = false
edition = "2021"

[dependencies]
arc-swap.workspace = true
fastcrypto.workspace = true
jsonrpsee.workspace = true
jsonrpsee-proc-macros.workspace = true
Expand Down
Loading

0 comments on commit ecb799c

Please sign in to comment.