-
Notifications
You must be signed in to change notification settings - Fork 13
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
RPC calls #13
RPC calls #13
Conversation
curl clean up added
Thank you for the PR! We defenitely need some built-in functions to interact with RPCs. But, since it's a C library, we should keep the code in C language. And, also, all the new functions must follow some rules (like each public function should start with the Using external libraries for serializing/deserializing JSON data? I don't know. I think it's better to make our own to keep the library as dependency free as possible. For making http calls, it's totally fine to use CURL. So, I can't merge this PR right now, but it stays open. Again, thanks for the PR! |
Hello bro...thanks for revising the PR. As u saied, the header i write have some dependencies on external json parser and curl. Also it is c++. It is very stable and it may use as a reference for the future if u want to include this feature. I am not very good at C but i will try to make a C version of this. |
Of course, I'll take inspirations from this PR. It's a great feature to have. |
ok bro 👍 .....i will code the header to fit the standard u specified. And also i was using nlohmann to parse previously. So to switch to this lib and make everything fit, it needs some time.... i will PR as soon as i finished it. Have a nice day bro 👍 |
Any progress on this? |
yes. But little straggle to fit in to the architecture u provided. The rest is progressing in good way 👍 . |
Hey bro. How are u. I'm not online this two day due to some personal problems. I think i will back tomorrow. Soryy for late🙏 |
Hey @DerXanRam . No worries, you don't have to. |
Hello mate. I'm back after 6 Hard days. How was everything? |
Hey @DerXanRam If you don't have free time to work on this, I'll merge the dev branch. |
Hello mate. do u mean integrating the |
@DerXanRam Yes, there are some other changes as well in that branch. I'll merge the |
ok mate..... merg it and i will PR when i finished it 👍 |
Created |
Closing the PR: #28 (comment) |
we can use this header like this in main.cpp
This header is tested againist infura.io and blastapi.io.
This headers and libraries are must be included to compile