Skip to content
This repository was archived by the owner on Dec 12, 2025. It is now read-only.

implemented worldname-check#1

Open
EinDev wants to merge 2 commits into
OurSources:masterfrom
EinDev:master
Open

implemented worldname-check#1
EinDev wants to merge 2 commits into
OurSources:masterfrom
EinDev:master

Conversation

@EinDev

@EinDev EinDev commented Jan 29, 2018

Copy link
Copy Markdown

should there be a limit for world name size?

@nagalun

nagalun commented Feb 19, 2018

Copy link
Copy Markdown
Contributor

Hey TimonMi, thanks for your contribution.
Late reply, but indeed, world names should be at least 1 character and not more than 24.
To avoid confusion and problems with some filesystems world names are always lowercase, it's the client's job to send the world name it wants to join in lowercase, that's why A-Z isn't checked in the for loop.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants