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 API Migration Guide for Horizon to RPC #1247

Merged
merged 8 commits into from
Feb 4, 2025

Conversation

leighmcculloch
Copy link
Member

@leighmcculloch leighmcculloch commented Feb 2, 2025

What

Adds a guide mapping Horizon REST API endpoints to RPC JSON-RPC API methods.

Screenshot 2025-02-03 at 1 09 34 pm

Why

Provides developers a clear reference for transitioning between APIs, detailing method mappings and migration strategies.

There don't exist docs today that help a developer who is already using Horizon know what endpoints they should look to on the RPC. While the endpoints are intuitive to a degree any migration comes with a cost, and documentation that helps devs see the path is clear or not can help with adoption.

The doc will also make it pretty quickly clear to a dev if they are relying on features of Horizon that just aren't supported by the RPC, so may help those devs start earlier at identifying other products and solutions they will need to acquire or build.

Most of the content in this page being added is derived from a diagram @mollykarcher and @ire-and-curses drew:

IMG_4562

There's not really a limit to how far or to what a migration guide could talk about. It could talk about operationally, or other aspects of the API such as pagination. But I think what's contained in this PR is at least a good starting point that should be impactful.

@stellar-jenkins
Copy link

@stellar-jenkins
Copy link

@leighmcculloch leighmcculloch marked this pull request as ready for review February 3, 2025 03:16
@leighmcculloch leighmcculloch requested review from a team, ElliotFriend and briwylde08 February 3, 2025 03:17
@stellar-jenkins
Copy link

2 similar comments
@stellar-jenkins
Copy link

@stellar-jenkins
Copy link

@stellar-jenkins
Copy link

@stellar-jenkins
Copy link

@leighmcculloch leighmcculloch merged commit 0f512b5 into main Feb 4, 2025
2 checks passed
@leighmcculloch leighmcculloch deleted the add-migrate-horizon-rpc-doc branch February 4, 2025 07:21
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.

4 participants