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

API v2 Updates #2841

Merged
merged 5 commits into from
Nov 26, 2024
Merged

API v2 Updates #2841

merged 5 commits into from
Nov 26, 2024

Conversation

oeggert
Copy link
Collaborator

@oeggert oeggert commented Nov 5, 2024

Code samples updates will be included in a separate PR.

@oeggert oeggert added the content updates Updating, cleaning up, or rewriting existing content for better usability or new contexts label Nov 5, 2024
@oeggert oeggert requested review from mDuo13 and amarantha-k November 5, 2024 00:29
@oeggert oeggert self-assigned this Nov 5, 2024
Copy link

socket-security bot commented Nov 5, 2024

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/[email protected] network +5 9.24 MB node-fetch-bot
pypi/[email protected] environment, eval, filesystem, network, shell, unsafe 0 19.4 MB reaperhulk
pypi/[email protected] environment, eval, filesystem, shell, unsafe 0 73 MB aclark, hugovk, radarhere, ...1 more

🚮 Removed packages: npm/[email protected]

View full report↗︎

Copy link
Collaborator

@amarantha-k amarantha-k left a comment

Choose a reason for hiding this comment

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

Other than one comment requesting clarification on the API versions used, the updates look good.


Currently, there are two API versions: `1` and `2` {% badge href="https://github.com/XRPLF/rippled/releases/tag/2.0.0" %}New in: rippled 2.0.0{% /badge %}. The server reports the range of supported API versions in the `version` API method. <!-- STYLE_OVERRIDE: will --> <!-- TODO: add a link when `version` method is documented. -->

Separate API requests can use different API versions even on the same persistent connection. For example, if you connect through WebSocket to a server that supports API versions 1 and 2, you can make an `account_tx` request using API version 2 and then make another `account_tx` request using API version 1 from the same connection.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Would be helpful to clarify a few things:

  • The default API versions used when requests are made through websockets (vs others like SDKs).
  • One can specify the API version you want to use for a specific request.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I moved the table up so the info about default API version used is more obvious. Also added a line about specifying version.

@oeggert oeggert requested a review from amarantha-k November 10, 2024 01:30
@tequdev tequdev added the needs japanese translation PRs with English changes that need to be added/updated in the Japanese translation of the site label Nov 12, 2024
Copy link
Collaborator

@amarantha-k amarantha-k left a comment

Choose a reason for hiding this comment

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

LGTM

@oeggert oeggert merged commit 75d3943 into master Nov 26, 2024
5 of 6 checks passed
@oeggert oeggert deleted the api-v2 branch November 26, 2024 18:37
@tequdev tequdev removed the needs japanese translation PRs with English changes that need to be added/updated in the Japanese translation of the site label Dec 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content updates Updating, cleaning up, or rewriting existing content for better usability or new contexts
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants