Skip to content

added customisation towards the notify setting in MiddleClickExtra#5934

Merged
Wide-Cat merged 3 commits into
MeteorDevelopment:masterfrom
RuttyboyzCode:master
Dec 18, 2025
Merged

added customisation towards the notify setting in MiddleClickExtra#5934
Wide-Cat merged 3 commits into
MeteorDevelopment:masterfrom
RuttyboyzCode:master

Conversation

@RuttyboyzCode

Copy link
Copy Markdown
Contributor

some servers uses different prefixes for msging players so i added a customizable prefix that isn't hardcoded

Type of change

  • Bug fix
  • [ ✔️] New feature

Description

A summary of the changes along with the reasoning behind the changes.

Related issues

Mention any issues that this pr relates to.

How Has This Been Tested?

Videos or screenshots of the changes if applicable.

Checklist:

  • [ ✔️] My code follows the style guidelines of this project.
  • [ ✔️] I have added comments to my code in more complex areas.
  • [ ✔️] I have tested the code in both development and production environments.

@noramibu

Copy link
Copy Markdown
Contributor

isn't it better to allow editing whole command (including message) instead of only editing command? so people can customize messages too

@RuttyboyzCode

Copy link
Copy Markdown
Contributor Author

Oh yeah, that's a much better idea

@RuttyboyzCode RuttyboyzCode changed the title added prefix to MiddleClickExtra added customisation towards the notify setting in MiddleClickExtra Dec 14, 2025
@DrParanoya

DrParanoya commented Dec 14, 2025

Copy link
Copy Markdown
Contributor

Typo on the description of the notifyMessage setting:
for the player's name should be without an apostrophe

@crosby-moe

Copy link
Copy Markdown
Collaborator

Typo on the description of the notifyMessage setting: for the player's name should be without an apostrophe

it looks correct to me

@DrParanoya

DrParanoya commented Dec 17, 2025

Copy link
Copy Markdown
Contributor

it looks correct to me

hmm I always thought there's no need for an apostrophe when writing stuff like that
guess there's no typo then

@Wide-Cat Wide-Cat merged commit c69800f into MeteorDevelopment:master Dec 18, 2025
MistressOfDNS pushed a commit to MistressOfDNS/meteor-client-fork that referenced this pull request May 7, 2026
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