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

Add Nodies as part of RPC Providers #1110

Merged
merged 3 commits into from
Dec 11, 2024
Merged

Conversation

nodiesBlade
Copy link
Contributor

@nodiesBlade nodiesBlade commented Dec 3, 2024

As part of the milestones for being a recipient of SCF #30, Nodies now supports both soroban and horizon endpoints. This PR aims to add us as a RPC provider for Stellar.

@ElliotFriend ElliotFriend self-requested a review December 5, 2024 17:16
@ElliotFriend
Copy link
Contributor

Hi, there @nodiesBlade!

Thanks for the contribution! Before I can get the PR approved and merged I tried testing out the RPC endpoints, and I couldn't get a response.

The mainnet endpoint gave me a "connection refused" error and the testnet endpoint gave a DNS error, like the hostname wasn't found. Can you double-check and confirm your RPC services are up and running, and I'll give it another try.

Thanks!

@nodiesBlade
Copy link
Contributor Author

nodiesBlade commented Dec 5, 2024

Hi, there @nodiesBlade!

Thanks for the contribution! Before I can get the PR approved and merged I tried testing out the RPC endpoints, and I couldn't get a response.

The mainnet endpoint gave me a "connection refused" error and the testnet endpoint gave a DNS error, like the hostname wasn't found. Can you double-check and confirm your RPC services are up and running, and I'll give it another try.

Thanks!

Hey @ElliotFriend , sorry! There was a small typo in the endpoints for the README. I've updated them now, can you try again

@briwylde08 briwylde08 self-requested a review December 11, 2024 18:45
@briwylde08 briwylde08 merged commit 6de2e44 into stellar:main Dec 11, 2024
1 check passed
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.

3 participants