Skip to content

Allows to display progress messages by player instead to everyone on the server.#312

Merged
katekerllenevich merged 8 commits into
PurpurMC:ver/1.16.5from
ExcaliaSI:advancement-message-by-player
May 3, 2021
Merged

Allows to display progress messages by player instead to everyone on the server.#312
katekerllenevich merged 8 commits into
PurpurMC:ver/1.16.5from
ExcaliaSI:advancement-message-by-player

Conversation

@Euphillya

Copy link
Copy Markdown
Contributor

When a server has a lot of people, player advancements spam the chat, a lot of servers turn it off to make the chat clean.
My patch allows to keep it on the server but by sending it only to the player concerned and not to the whole server.

Comment thread patches/server/0206-Allows-to-display-progress-messages-by-player-instea.patch Outdated
Comment thread patches/server/0206-Allows-to-display-progress-messages-by-player-instea.patch Outdated
Comment thread patches/server/0206-Allows-to-display-progress-messages-by-player-instea.patch Outdated
Comment thread patches/server/0206-Allows-to-display-progress-messages-by-player-instea.patch Outdated
Comment thread patches/server/0206-Allows-to-display-progress-messages-by-player-instea.patch Outdated
Comment thread patches/server/0206-Allows-to-display-progress-messages-by-player-instea.patch Outdated
Comment thread patches/server/0206-Allows-to-display-progress-messages-by-player-instea.patch Outdated
@Euphillya Euphillya requested a review from katekerllenevich May 3, 2021 11:21
Comment thread patches/server/0206-Allows-to-display-progress-messages-by-player-instea.patch Outdated
Comment thread patches/server/0206-Allows-to-display-progress-messages-by-player-instea.patch Outdated
@Euphillya Euphillya requested a review from katekerllenevich May 3, 2021 15:06
@Euphillya

Copy link
Copy Markdown
Contributor Author

It's good

@katekerllenevich katekerllenevich merged commit 8b2607c into PurpurMC:ver/1.16.5 May 3, 2021
@Euphillya Euphillya deleted the advancement-message-by-player branch June 12, 2021 16:50
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.

3 participants