-
Notifications
You must be signed in to change notification settings - Fork 16
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
service/state: Provide simple scaffold for
StateService
and impleme…
…nt `CoreAccess` (#420) * CoreAccessor * update ADR with changes made to CoreAccess design * cleanup * use new insecure.NewCredentials w grpc cli * keyconf documentation * add doc.go * document service struct * remove unnecessary type aliases * add lifecycle hooks * test: failing due to cel-app panic with InitChain * chore|refactor: go mod tidy & change grpc dial to DialContext * fix: add watchdog dep * chore: lint * refactor: dont return err with constructor * docs: update state ADR to remove mentions of lens * fix: remove unused encCfg field from CoreAccess * refactor: remove KeyConfig and use "default" as KeyringAccName by default * chore: mod tidy * refactor: setup queryClient in Start of CoreAccess * docs: fix typo in lightComponents doc * comment out test * chore: lint * bug: grpc needs insecure credentials * refactor: BalanceForAddress takes types.AccAddress instead of string, state access available on full node too * refactor: use BroadcastMode_BROADCAST_MODE_BLOCK as default for tx submission * feat: add SubmitTxWithBroadcastMode but dont add to StateAccessor interface for now * refactor: pass keystore to constructor instead of string to path and implement Path method in Keystore | lint * chore: update celestia-core dep to prevent weird lint error * refactor: add state serv to baseComponents * refactor: BalanceForAddress takes a types.Address from cosmos sdk instead of types.AccAddress * doc: fix typo Co-authored-by: Hlib Kanunnikov <hlibwondertan@gmail.com> * test: delete unit test for core access as it doesnt work in favour of celestiaorg/celestia-node#570 * refactor: add params.Network after rebase Co-authored-by: Hlib Kanunnikov <hlibwondertan@gmail.com>
- Loading branch information
Showing
2 changed files
with
587 additions
and
36 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.