Skip to content

XLS-33d: Compact Fungible Tokens (CFTs)#109

Merged
intelliot merged 7 commits intoXRPLF:masterfrom
ledhed2222:XLS-33
Jun 22, 2023
Merged

XLS-33d: Compact Fungible Tokens (CFTs)#109
intelliot merged 7 commits intoXRPLF:masterfrom
ledhed2222:XLS-33

Conversation

@ledhed2222
Copy link
Copy Markdown
Contributor

@ledhed2222 ledhed2222 commented May 15, 2023

Update: This PR has been merged and the XLS-33d spec can be viewed here.

Please feel free to open a new PR to propose any changes or updates to the spec.


Initial commit of XLS-33d spec

Per CONTRIBUTING.md - this is ready to merge as a draft spec. The spec will continue to be updated with new PRs after this!

@sappenin sappenin self-requested a review May 17, 2023 16:07
@ledhed2222 ledhed2222 marked this pull request as ready for review May 19, 2023 20:37
@ledhed2222 ledhed2222 requested review from Silkjaer and intelliot May 19, 2023 20:39
@sappenin sappenin self-requested a review May 19, 2023 20:49
Copy link
Copy Markdown
Collaborator

@sappenin sappenin left a comment

Choose a reason for hiding this comment

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

This LGTM as a DRAFT.

@sappenin
Copy link
Copy Markdown
Collaborator

+1 to all of the changes proposed by @intelliot (I can't merge them though).

:)

Copy link
Copy Markdown
Contributor

@Silkjaer Silkjaer left a comment

Choose a reason for hiding this comment

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

A few fixes needed before it can be merged :)

@ledhed2222 ledhed2222 requested a review from shawnxie999 May 30, 2023 17:55
@intelliot intelliot changed the title XLS-33 draft XLS-33d: Compact Fungible Tokens (CFTs) May 31, 2023
@intelliot intelliot removed their assignment Jun 6, 2023
@intelliot
Copy link
Copy Markdown
Contributor

@ledhed2222 please respond to the above comments, at your convenience

@ledhed2222
Copy link
Copy Markdown
Contributor Author

@intelliot done :)

@sappenin
Copy link
Copy Markdown
Collaborator

@Silkjaer - all requested changes have been merged by @ledhed2222. Once you give a final approval I'll go ahead and merge this one.

@intelliot intelliot mentioned this pull request Jun 20, 2023
@ledhed2222
Copy link
Copy Markdown
Contributor Author

@Silkjaer hey there! can you review again please?

Copy link
Copy Markdown
Contributor

@Silkjaer Silkjaer left a comment

Choose a reason for hiding this comment

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

LGTM!

@intelliot intelliot merged commit ce29208 into XRPLF:master Jun 22, 2023
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.

5 participants