Skip to content

Conversation

mDuo13
Copy link
Collaborator

@mDuo13 mDuo13 commented Oct 7, 2025

Fix #2728.

  • Correct the description of this flag in the RippleState reference and put it into the correct order (the table is sorted by numeric value of the flags, ascending)
  • Add proper amendment disclaimers for lsfAMMNode and the two DeepFreeze flags in the RippleState reference.
  • Improve the whitespace alignment of the RippleState flags table (in the md source).
  • Move the details of checking for lsfAMMNode flag from the AMMs concept page (where I felt that it was too much detail for that level) to the "Look Up Transaction Results" page (which is a much better fit for this type of thorough detail, although the rest of the page is slightly out of date) and linked there from the AMMs page.
  • Updated some things for style in the areas around where I was already editing, like changing "Offer" to "offer" unless specifically talking about an Offer ledger entry, saying "ledger entry" instead of "ledger object", and referring to "trust line tokens" to distinguish from MPTs.

@mDuo13 mDuo13 added content updates Updating, cleaning up, or rewriting existing content for better usability or new contexts defi Relating to the DEX, currency/asset trading, market-making/AMM, and other on-chain financial tools labels Oct 7, 2025
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 defi Relating to the DEX, currency/asset trading, market-making/AMM, and other on-chain financial tools

Projects

None yet

Development

Successfully merging this pull request may close these issues.

AMM docs: Clarification about the usage of lsfAMMNode flag

2 participants