Skip to content

Feat/initialize tokens from chain denoms - #286

Merged
aarmoa merged 14 commits into
devfrom
feat/initialize_tokens_from_chain_denoms
Jan 2, 2024
Merged

Feat/initialize tokens from chain denoms#286
aarmoa merged 14 commits into
devfrom
feat/initialize_tokens_from_chain_denoms

Conversation

@aarmoa

@aarmoa aarmoa commented Dec 26, 2023

Copy link
Copy Markdown
Contributor
  • Implemented all missing messages in chain Bank module
  • Added logic in AsyncClient to initialize tokens from the chain denoms metadata

@aarmoa
aarmoa requested a review from achilleas-kal December 26, 2023 13:16
@codecov

codecov Bot commented Dec 26, 2023

Copy link
Copy Markdown

Codecov Report

Attention: 20 lines in your changes are missing coverage. Please review.

Comparison is base (2aeca6a) 72.00% compared to head (7143944) 72.22%.
Report is 6 commits behind head on dev.

Files Patch % Lines
pyinjective/async_client.py 64.28% 11 Missing and 4 partials ⚠️
...injective/client/chain/grpc/chain_grpc_bank_api.py 89.36% 1 Missing and 4 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##              dev     #286      +/-   ##
==========================================
+ Coverage   72.00%   72.22%   +0.22%     
==========================================
  Files          46       46              
  Lines        3150     3237      +87     
  Branches      284      295      +11     
==========================================
+ Hits         2268     2338      +70     
- Misses        814      826      +12     
- Partials       68       73       +5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@aarmoa
aarmoa merged commit ffd1928 into dev Jan 2, 2024
@aarmoa
aarmoa deleted the feat/initialize_tokens_from_chain_denoms branch January 2, 2024 13:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants