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

Masternode RPC (createmasternode, listmasternodes, getmasternode) #372

Merged
merged 34 commits into from
Jun 23, 2021

Conversation

siradji
Copy link
Contributor

@siradji siradji commented Jun 11, 2021

What kind of PR is this?:

/kind feature

What this PR does / why we need it:

Masternode RPC implementation #48

Which issue(s) does this PR fixes?:

Fixes part of #48

Additional comments?:

@codeclimate
Copy link

codeclimate bot commented Jun 11, 2021

Code Climate has analyzed commit 03a93f8 and detected 0 issues on this pull request.

View more on Code Climate.

@netlify
Copy link

netlify bot commented Jun 11, 2021

✔️ Deploy Preview for jellyfish-defi ready!

🔨 Explore the source changes: 03a93f8

🔍 Inspect the deploy log: https://app.netlify.com/sites/jellyfish-defi/deploys/60d33cfe2ddcf4000861bf0f

😎 Browse the preview: https://deploy-preview-372--jellyfish-defi.netlify.app

@github-actions
Copy link

github-actions bot commented Jun 11, 2021

size-limit report 📦

Path Size Loading time (3g) Running time (snapdragon) Total time
packages/jellyfish/dist/index.umd.js 20.82 KB (+1.27% 🔺) 417 ms (+1.27% 🔺) 187 ms (+7.15% 🔺) 603 ms

@jingyi2811
Copy link
Contributor

You need to update sidebar.js too

@siradji
Copy link
Contributor Author

siradji commented Jun 11, 2021

You need to update sidebar.js too

Okay. Updated now.

Copy link
Contributor

@canonbrother canonbrother left a comment

Choose a reason for hiding this comment

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

just some missing props and minor refactors

thumbs up for your code look cleaner and organized

packages/jellyfish-api-core/src/category/masternode.ts Outdated Show resolved Hide resolved
packages/jellyfish-api-core/src/category/masternode.ts Outdated Show resolved Hide resolved
packages/jellyfish-api-core/src/category/masternode.ts Outdated Show resolved Hide resolved
packages/jellyfish-api-core/src/category/masternode.ts Outdated Show resolved Hide resolved
packages/jellyfish-api-core/src/category/masternode.ts Outdated Show resolved Hide resolved
@siradji
Copy link
Contributor Author

siradji commented Jun 11, 2021

just some missing props and minor refactors

thumbs up for your code look cleaner and organized

@canonbrother All thanks to you guys for getting some sense into me. Keep it up till i can walk straight!

@siradji siradji marked this pull request as ready for review June 21, 2021 10:32
@siradji siradji requested a review from fuxingloh June 21, 2021 10:33
@fuxingloh fuxingloh marked this pull request as draft June 22, 2021 04:54
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.

generics code improvements, rmb to change the docs

packages/jellyfish-api-core/src/category/masternode.ts Outdated Show resolved Hide resolved
packages/jellyfish-api-core/src/category/masternode.ts Outdated Show resolved Hide resolved
packages/jellyfish-api-core/src/category/masternode.ts Outdated Show resolved Hide resolved
packages/jellyfish-api-core/src/category/masternode.ts Outdated Show resolved Hide resolved
packages/jellyfish-api-core/src/category/masternode.ts Outdated Show resolved Hide resolved
packages/jellyfish-api-core/src/category/masternode.ts Outdated Show resolved Hide resolved
@siradji siradji marked this pull request as ready for review June 23, 2021 11:33
@siradji siradji requested a review from fuxingloh June 23, 2021 11:44
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.

1 minor mistake, I will edit it

website/docs/jellyfish/api/masternode.md Outdated Show resolved Hide resolved
@fuxingloh fuxingloh merged commit 58fba5e into main Jun 23, 2021
@fuxingloh fuxingloh deleted the siradji/rpc_masternodes branch June 23, 2021 14:11
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.

6 participants