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.

Sign up for GitHub

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

Jump to bottom

Remove "minecraft:" prefix to allow for longer messages #13

Closed
Balint817 opened this issue Jun 28, 2023 · 2 comments · Fixed by #14
Closed

Remove "minecraft:" prefix to allow for longer messages #13

Balint817 opened this issue Jun 28, 2023 · 2 comments · Fixed by #14
Labels
enhancement New feature or request

Comments

@Balint817
Copy link

Title + in case something other than minecraft: is present (modname:), then simply do nothing
@jaxvanyang
Copy link
Owner

Title + in case something other than minecraft: is present (modname:), then simply do nothing

Hi! I didn't know that the namespace can be omitted. You remind me that the @s in tp @s x y z can also be omitted.

But this kind of trick only increase the length a little. I will update for this. But if you have a very long description need, you can first send the clickable message for teleporting, and then send a regular message follows that.

@jaxvanyang jaxvanyang added the enhancement New feature or request label Jun 29, 2023
jaxvanyang added a commit that referenced this issue Jun 29, 2023
@jaxvanyang
feat: reduce command length for longer description
6191495 
- remove unnecessary "minecraft:" prefix
- remove unnecessary "@s" in /tp command
- update README

Close #13
@jaxvanyang jaxvanyang mentioned this issue Jun 29, 2023
Merged
jaxvanyang added a commit that referenced this issue Jun 29, 2023
@jaxvanyang
feat: reduce command length for longer description
12f74f0 
- remove unnecessary "minecraft:" prefix
- remove unnecessary "@s" in /tp command
- update README

Close #13
@jaxvanyang
Copy link
Owner

@Balint817 This update has been published on Modrinth, hope you enjoy it. 😉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: reduce command length for longer description jaxvanyang/tp-chat-cmd
2 participants
@jaxvanyang @Balint817