Skip to content

Sorts worldname by alphabetical order#2432

Closed
benwoo1110 wants to merge 1 commit into
Multiverse:mainfrom
benwoo1110:sorted-worldlist
Closed

Sorts worldname by alphabetical order#2432
benwoo1110 wants to merge 1 commit into
Multiverse:mainfrom
benwoo1110:sorted-worldlist

Conversation

@benwoo1110

@benwoo1110 benwoo1110 commented Oct 6, 2020

Copy link
Copy Markdown
Member

Make MVWorld implement Comparable interface.

Had to add new WorldManager#getSortedMVWorlds as return type is different.

fixes #1960

Make MVWorld implement Comparable interface.

Had to add new WorldManager#getSortedMVWorlds as return type is different.
@benwoo1110

Copy link
Copy Markdown
Member Author

image
Alphabetical order!

@dumptruckman dumptruckman added this to the Multiverse 5.0.0 milestone Oct 8, 2020
@benwoo1110 benwoo1110 added the PR: Enhancement Pull requests to implement a feature or improvement in code. label Oct 9, 2020
@nicegamer7 nicegamer7 changed the base branch from master to main November 25, 2020 18:34
@benwoo1110 benwoo1110 added the State: Needs Review By Dev Pull requests requires the approve of lead dev. label Nov 27, 2020
@benwoo1110 benwoo1110 mentioned this pull request Dec 20, 2020
16 tasks

@nicegamer7 nicegamer7 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM 👍.

@benwoo1110 benwoo1110 removed this from the Multiverse 5.0.0 milestone Feb 18, 2021
@benwoo1110

Copy link
Copy Markdown
Member Author

I think this one let use command display, cant rmb what I coded 9 month ago anyways

@benwoo1110 benwoo1110 closed this Jul 16, 2021
@SlimeDog

Copy link
Copy Markdown

Is this going to be added to MV 4.x ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: Enhancement Pull requests to implement a feature or improvement in code. State: Needs Review By Dev Pull requests requires the approve of lead dev.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Feature request: Order mv list output alphabetically

4 participants