Skip to content

Remove Dynamic MPT clear flags + minor updates#217

Open
maria-robobug wants to merge 2 commits into
mainfrom
remove-clear-flags-dynamic-mpt
Open

Remove Dynamic MPT clear flags + minor updates#217
maria-robobug wants to merge 2 commits into
mainfrom
remove-clear-flags-dynamic-mpt

Conversation

@maria-robobug

@maria-robobug maria-robobug commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

This PR aligns the documentation with the spec changes in XRPL-Standards PR #562. The spec now states mutable MPT issuance flags are strictly one-way: the tmfMPTCanMutateCan* flags were renamed to tmfMPTCanEnable* to reflect that they only authorize enabling (not toggling), and the corresponding tmfMPTClear* flags were removed from MPTokenIssuanceSet entirely.

  • Minor concept doc updates.
  • Update reference docs.

@maria-robobug maria-robobug marked this pull request as ready for review June 8, 2026 09:21
@maria-robobug maria-robobug changed the title Remove Dynamic MPT clear flags (except for RequireAuth) Remove Dynamic MPT clear flags + minor updates Jun 11, 2026
@maria-robobug maria-robobug force-pushed the remove-clear-flags-dynamic-mpt branch from 3817395 to bf6c065 Compare June 11, 2026 13:32

@oeggert oeggert left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Two minor issues found, but otherwise good.

Comment thread docs/xls-94-dynamic-mpts/reference.md
Comment thread docs/xls-94-dynamic-mpts/reference.md Outdated
Co-authored-by: oeggert <117319296+oeggert@users.noreply.github.com>
@maria-robobug maria-robobug force-pushed the remove-clear-flags-dynamic-mpt branch from d590b19 to ed18041 Compare June 12, 2026 08:47
@maria-robobug maria-robobug requested a review from oeggert June 12, 2026 08:47

@oeggert oeggert left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

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.

2 participants