Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

filename seperator setting to fix #54 #83

Merged
merged 5 commits into from May 10, 2023

Conversation

blaboing
Copy link
Contributor

Adds a filename split seperator setting that lets you choose between old style _ and . because dot is the only char that isn´t allowed in playername but in texturenames.
Default value is _ to keep it compatible with older versions, while . offers a solution to fix #54.

@ronoaldo
Copy link

This looks good. I'll test it out!

Copy link
Member

@SmallJoker SmallJoker left a comment

Choose a reason for hiding this comment

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

LGTM. Will merge in a few days unless there's objections.

@SmallJoker SmallJoker merged commit d3c7fa7 into minetest-mods:master May 10, 2023
SmallJoker pushed a commit that referenced this pull request May 10, 2023
Adds a filename split seperator setting that lets you choose between old style `_` and `.` because dot is the only char that isn´t allowed in playername but in texturenames.
Default value is `_` to keep it compatible with older versions, while `.` offers a solution to fix #54.
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.

Private skin for players with underscore in the name does not work
3 participants