📝 Refresh List of Operations - #2754
Conversation
Co-authored-by: Codex <noreply@openai.com>
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Updates the transactions “list of operations” doc to improve readability by standardizing headings and tightening several operation descriptions, including deprecation guidance.
Changes:
- Standardized operation headings to Title Case and added consistent punctuation.
- Rewrote multiple operation summaries/parameter descriptions for clarity and added cross-links.
- Replaced the
Allow Trustdeprecation warning block with an inline deprecation sentence.
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| | `REVOKE_SPONSORSHIP_LEDGER_ENTRY` | LedgerKey | ledgerKey | Ledger key that holds information to identify a specific ledgerEntry that may have its sponsorship modified. See [LedgerKey](../../glossary.mdx#ledgerkey) for more information. | | ||
|
|
||
| Or | ||
| or |
There was a problem hiding this comment.
I disagree. It's not really a standalone label but actually an enumeration for the possible types, continuing the introductory sentence at 524. Both of the tables can be thought of as objects referenced by the list, which terminates at the 'possible errors' header.
Related pathfinding documentation issue: stellar#1545
|
Warning This PR is blocked from merging until PR #2398 is merged. |
from 723
rel #2579 (comment)