tari-project / tari-dan

BSD 3-Clause "New" or "Revised" License
7 stars 16 forks source link

refactor(wallet_daemon): use the indexer for substate listing #1060

Closed mrnaveira closed 3 months ago

mrnaveira commented 3 months ago

Description

Motivation and Context

On https://github.com/tari-project/tari-dan/pull/1049 we added a new indexer JRPC method for listing substates. The indexer substate listing is more advanced than the one in the wallet daemon:

This PR changes the substates.list JRPC method in the wallet daemon to proxy the call to the corresponding method in the indexer. It also adds pagination support.

How Has This Been Tested?

Manually by:

What process can a PR reviewer use to test or verify this change?

See previous sections

Breaking Changes

github-actions[bot] commented 3 months ago

Test Results (CI)

537 tests   - 1   537 :white_check_mark:  - 1   1h 38m 42s :stopwatch: - 21m 31s  64 suites ±0     0 :zzz: ±0    2 files   ±0     0 :x: ±0 

Results for commit cfbb6e24. ± Comparison against base commit f61dcd47.

This pull request removes 2 and adds 1 tests. Note that renamed tests count towards both. ``` tari_dan_wallet_sdk ‑ models::substate::export_bindings_substatetype tari_indexer_client ‑ types::export_bindings_substatetype ``` ``` tari_dan_common_types ‑ substate_type::export_bindings_substatetype ```