Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

account.listaccounts, account.getaccount, account.gettokenbalances, account.listaccounthistory #162

Conversation

canonbrother
Copy link
Contributor

@canonbrother canonbrother commented Apr 26, 2021

What kind of PR is this?:

/kind feature

What this PR does / why we need it:

Implemented type for RPC: #48

  1. account.listaccounts
  2. account.getaccount
  3. account.gettokenbalances
  4. account.listaccounthistory

@github-actions
Copy link

github-actions bot commented Apr 26, 2021

size-limit report 📦

Path Size Loading time (3g) Running time (snapdragon) Total time
packages/jellyfish/dist/jellyfish.umd.js 18.76 KB (+1.36% 🔺) 376 ms (+1.36% 🔺) 178 ms (-1.77% 🔽) 553 ms

@canonbrother canonbrother marked this pull request as draft April 26, 2021 16:03
@fuxingloh fuxingloh added this to the Jellyfish Documentation milestone Apr 26, 2021
@netlify
Copy link

netlify bot commented Apr 27, 2021

Deploy preview for jellyfish-defi ready!

Built with commit ba53570

https://deploy-preview-162--jellyfish-defi.netlify.app

@codecov
Copy link

codecov bot commented Apr 28, 2021

Codecov Report

Merging #162 (255ef4b) into main (a279db6) will increase coverage by 0.03%.
The diff coverage is 95.45%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #162      +/-   ##
==========================================
+ Coverage   95.45%   95.48%   +0.03%     
==========================================
  Files          53       55       +2     
  Lines        1122     1174      +52     
  Branches      136      149      +13     
==========================================
+ Hits         1071     1121      +50     
- Misses         49       50       +1     
- Partials        2        3       +1     
Impacted Files Coverage Δ
...ackages/jellyfish-api-core/src/category/account.ts 94.73% <94.73%> (ø)
packages/jellyfish-api-core/src/index.ts 100.00% <100.00%> (ø)
packages/testcontainers/src/index.ts 100.00% <0.00%> (ø)
...ainers/src/chains/reg_test_container/persistent.ts 95.45% <0.00%> (ø)
packages/testcontainers/src/chains/container.ts 88.57% <0.00%> (+0.10%) ⬆️
...ainers/src/chains/reg_test_container/masternode.ts 98.00% <0.00%> (+0.17%) ⬆️
packages/jellyfish-crypto/src/bech32.ts 95.00% <0.00%> (+0.55%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a279db6...255ef4b. Read the comment docs.

@canonbrother canonbrother marked this pull request as ready for review April 29, 2021 01:05
Copy link
Contributor

@fuxingloh fuxingloh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

great work, I wrote some minor changes to consider

website/docs/jellyfish/api/account.md Outdated Show resolved Hide resolved
packages/jellyfish-api-core/src/category/account.ts Outdated Show resolved Hide resolved
packages/jellyfish-api-core/src/category/account.ts Outdated Show resolved Hide resolved
packages/jellyfish-api-core/src/category/account.ts Outdated Show resolved Hide resolved
Comment on lines +55 to +59
await container.call('createtoken', [metadata])
await container.generate(25)

await container.call('minttokens', [`${amount.toString()}@${symbol}`])
await container.generate(25)
Copy link
Contributor

@fuxingloh fuxingloh Apr 29, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

related to this DeFiCh/ain#346 that you reported, thanks!

…accounts as a func(), rpc: use optional params instead of defaut to fix coverage issue
@fuxingloh fuxingloh merged commit a969279 into main Apr 30, 2021
@fuxingloh fuxingloh deleted the canonbrother/rpc-accounts-listaccounts-getaccount-gettokenbalances-listaccounthistory branch April 30, 2021 10:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature New feature request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants