Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Comments

fix: add silence leave for premiumvanish#168

Open
ChipWolf wants to merge 1 commit intoMultiChat:master from
ChipWolf:master
Open

fix: add silence leave for premiumvanish #168
ChipWolf wants to merge 1 commit intoMultiChat:master from
ChipWolf:master

Conversation

@ChipWolf
Copy link

@ChipWolf ChipWolf commented Feb 26, 2021
edited
Loading

Description

What, briefly, is the purpose of this pull request?

Adds functionality to silence leave messages if the user is vanished with PremiumVanish.

Is this pull request related to an issue with the plugin?

Fixes #167

What is the status of this pull request? [e.g. In Development, Undergoing Final Testing, Ready]

Ready for merge

Changes

What currently existing parts of the plugin are affected by this pull request

If users vanish with PremiumVanish, it sends a fake leave message. If they then leave the server, there's another leave message. Users which are vanished with PremiumVanish cannot leave incognito.

What new features are added to the plugin by this pull request

An option to silence leave messages if a user is vanished with PremiumVanish.

Standards

Does this pull request adhere to basic Java coding standards?

I matched the standard of the existing code implemented to silence the join messages.

Does this pull request seem to stay true to the style used so far in the MultiChat code.

Yes, it's essentially a copy and paste job :)

Copy link
Contributor

@akshualy akshualy left a comment

Choose a reason for hiding this comment

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

Change comment & should be fine.

Copy link
Author

@OllieMartin ready for you to merge

Copy link
Author

@Gadsee do you have merge access?

@ChipWolf ChipWolf changed the title (削除) feat: add silence leave for premiumvanish (削除ここまで) (追記) fix: add silence leave for premiumvanish (追記ここまで) Apr 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@akshualy akshualy akshualy left review comments

@OllieMartin OllieMartin Awaiting requested review from OllieMartin

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Quitmessages are visible even when premiumvanish vanishmode active

AltStyle によって変換されたページ (->オリジナル) /