Skip to content

XLS-94 Update: remove clear mutable flags for dynamic MPT#562

Open
yinyiqian1 wants to merge 10 commits into
XRPLF:masterfrom
yinyiqian1:dynamic_mpt_new
Open

XLS-94 Update: remove clear mutable flags for dynamic MPT#562
yinyiqian1 wants to merge 10 commits into
XRPLF:masterfrom
yinyiqian1:dynamic_mpt_new

Conversation

@yinyiqian1

@yinyiqian1 yinyiqian1 commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

High Level Overview of Change

We have a spec change for dynamic MPT disallowing clear mutable flags.

Context of Change

  • All the tmfMPTClear mutable flags are removed. Because we only allow the issuance flags being enabled once if they were set as mutable during issuance, and then they will remain immutable.
  • Since tmfMPTClear mutable flags are removed, the bit value for tmfMPTSet mutable flags are updated.
  • CanMutate flags are renamed to CanEnable

Type of Change

  • New XLS Draft
  • XLS Update (changes to an existing XLS)
  • XLS Status Change (e.g., Draft → Final, Draft → Stagnant)
  • Process/Meta (changes to CONTRIBUTING.md, XLS-1, templates, etc.)
  • Infrastructure (CI, workflows, scripts, website)
  • Documentation (README updates, typo fixes)

@yinyiqian1 yinyiqian1 marked this pull request as ready for review June 9, 2026 21:32
@yinyiqian1 yinyiqian1 changed the title Remove clear mutable flags for dynamic MPT XLS-94 Update: remove clear mutable flags for dynamic MPT Jun 9, 2026
Comment thread XLS-0094-dynamic-MPT/README.md Outdated
Comment thread XLS-0094-dynamic-MPT/README.md
Comment thread XLS-0094-dynamic-MPT/README.md Outdated

@Kassaking7 Kassaking7 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@kennyzlei kennyzlei requested a review from Tapanito June 25, 2026 21:25
@kennyzlei kennyzlei requested a review from mvadari July 9, 2026 21:04
Comment thread XLS-0094-dynamic-MPT/README.md Outdated
| `MutableFlags` attempts to modify flags not declared as mutable | `tecNO_PERMISSION` |
| `MPTokenMetadata` is present but was not marked as mutable | `tecNO_PERMISSION` |
| `TransferFee` is present but was not marked as mutable | `tecNO_PERMISSION` |
| Failure Condition | Error Code |

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.

Please format this as a numbered list - e.g.

1. `MutableFlags` contains invalid value (0 is invalid as well) (`temINVALID_FLAG`)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@yinyiqian1 yinyiqian1 requested a review from mvadari July 10, 2026 16:55
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.

3 participants