Skip to content

Add getmininginfo call#228

Open
BHodl wants to merge 5 commits into
SatoshiPortal:devfrom
BHodl:features/getmininginfo
Open

Add getmininginfo call#228
BHodl wants to merge 5 commits into
SatoshiPortal:devfrom
BHodl:features/getmininginfo

Conversation

@BHodl

@BHodl BHodl commented Aug 23, 2021

Copy link
Copy Markdown
Contributor

Let me know if I miss something. Not tested!

@BHodl

BHodl commented Aug 26, 2021

Copy link
Copy Markdown
Contributor Author

The getmininginfo call tested and working. The getblockhashps call don't pass the gatekeeper. Maybe related to api.properties, wip.

@BHodl

BHodl commented Aug 26, 2021

Copy link
Copy Markdown
Contributor Author

Fixed but.. Currently getblockhashps with optional blockheight not returning the right hashps. Let wait the node is fully synced to see...
Maybe we shoudl add a send_to_stats_node function for that kind of call, not including wallet specifics.

@Kexkey

Kexkey commented Sep 1, 2021

Copy link
Copy Markdown
Collaborator

Thanks and sorry for the delay, have been particularly busy lately.

Did you get a chance to make it work?

@BHodl

BHodl commented Sep 1, 2021

Copy link
Copy Markdown
Contributor Author

Yeah! No problemo, we all want to see lnurl stuff in cyphernode, keep up the good work! The getmininginfo is working perfectly, but things need to be modified for the getblockhashps. I will probably change the call name for getnetworkhashps to be consistent with bitcoincore call, and modify the returned data. I wait my cyphernode to be fully sync to test that one. I will confirm when fully tested.

@BHodl

BHodl commented Sep 8, 2021

Copy link
Copy Markdown
Contributor Author

Fully tested and working! It's my first PR so let me know if I miss something. I'm mostly setup for testing changes now, next time I will probably do my coding and testing before pushing the PR. One thing that can be improved is adding a send_to_stats_node function in sendtobitcoinnode.sh not including wallets specifics for stats call. But since it's affect all stats call, a separate PR would be better. Let me know what you think!

@BHodl

BHodl commented Mar 22, 2022

Copy link
Copy Markdown
Contributor Author

Updated to solve v0.8.0 conflicts. Tested and working.

Comment thread doc/openapi/v0/cyphernode-api.yaml
Comment thread proxy_docker/app/script/blockchainrpc.sh Outdated
Comment thread proxy_docker/app/script/blockchainrpc.sh Outdated
Comment thread proxy_docker/app/script/requesthandler.sh
Comment thread proxy_docker/app/script/tests.sh Outdated
@BHodl BHodl force-pushed the features/getmininginfo branch from d90cd3f to 4c80f4b Compare March 24, 2022 03:23
@Kexkey

Kexkey commented Mar 24, 2022

Copy link
Copy Markdown
Collaborator

Rebase on dev, thanks!

@BHodl BHodl force-pushed the features/getmininginfo branch from 9742e34 to 50584d6 Compare March 24, 2022 23:17
@BHodl BHodl changed the base branch from master to dev March 24, 2022 23:23
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