Skip to content

Conversation

@maria-robobug
Copy link
Contributor

@maria-robobug maria-robobug commented Oct 17, 2025

Issue #3338. See related XLS PR.

  • Updates MPT Concept docs with metadata schema
  • Updates issue-mpt-with-metdata example code with new schema changes.

Preview: https://xrpl-dev-portal--mpt-metadata-schema.preview.redocly.app/docs/concepts/tokens/fungible-tokens/multi-purpose-tokens#on-chain-metadata

- Updates MPT Concept docs with metadata schema
- Updates issue-mpt-with-metdata example code with new schema changes.
Copy link
Collaborator

@mDuo13 mDuo13 left a comment

Choose a reason for hiding this comment

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

Looks ok though I had something of an observation/complaint.

|:--------------- |:---- |:---------------- |---------- |-------------|
| ticker | `t` | String | Yes | The ticker symbol used to represent the token. Must be uppercase letters (A-Z) and digits (0-9) only. A maximum of 6 characters is recommended. |
| name | `n` | String | Yes | The display name of the token. Any UTF-8 string is permitted. |
| desc | `d` | String | No | A short description of the token. Any UTF-8 string is permitted. |
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is also a problem with how XLS-89 evolved, but it seems more confusing than useful to present a "Field Name" that isn't used before the (shorter) "Key" that actually is used, but doubly so when the longer "Field Name" is already an abbreviation like "desc"

@maria-robobug maria-robobug merged commit 31ff09c into master Oct 23, 2025
4 of 5 checks passed
@maria-robobug maria-robobug deleted the mpt-metadata-schema branch October 23, 2025 16:09
maria-robobug added a commit that referenced this pull request Oct 28, 2025
Add MPT Metadata Schema section to concept docs
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