Skip to content

Comments

ledger_entry updates#3418

Merged
oeggert merged 22 commits intomasterfrom
ledger-entry-updates
Jan 29, 2026
Merged

ledger_entry updates#3418
oeggert merged 22 commits intomasterfrom
ledger-entry-updates

Conversation

@oeggert
Copy link
Collaborator

@oeggert oeggert commented Dec 11, 2025

Adding missing entries, cleaning up common links, and fixing miscellaneous issues.

TO DO:

  • Alphabetize ledger_entry.md and command-list.json.

@oeggert oeggert marked this pull request as ready for review December 18, 2025 05:59
oeggert and others added 5 commits December 19, 2025 14:50
…methods/ledger_entry.md

Co-authored-by: Maria Shodunke <maria-robobug@users.noreply.github.com>
…methods/ledger_entry.md

Co-authored-by: Maria Shodunke <maria-robobug@users.noreply.github.com>
…methods/ledger_entry.md

Co-authored-by: Maria Shodunke <maria-robobug@users.noreply.github.com>
…methods/ledger_entry.md

Co-authored-by: Maria Shodunke <maria-robobug@users.noreply.github.com>
…methods/ledger_entry.md

Co-authored-by: Maria Shodunke <maria-robobug@users.noreply.github.com>
- [Get MPToken Entry](#get-mptoken-entry)
- [Response Format](#response-format)
- [Possible Errors](#possible-errors)
In addition to the general fields above, you must specify _exactly 1_ of the following fields to indicate what type of entry to retrieve:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Seems wrong to say "one of the following" with the paragraph ending in a colon and then simply proceed to the next heading...

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 see what you mean. I tweaked the intro and moved this line directly into the Request Format section.

@oeggert oeggert requested a review from mDuo13 January 17, 2026 04:52
Copy link
Contributor

@maria-robobug maria-robobug left a comment

Choose a reason for hiding this comment

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

LGTM, the failing links are related to SAV links that don't exist yet (pending review).

@oeggert oeggert merged commit 1049771 into master Jan 29, 2026
3 of 5 checks passed
@oeggert oeggert deleted the ledger-entry-updates branch January 29, 2026 18:39
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.

Undocumented ledger_entry fields Update documentation for ledger_entry Request

3 participants