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

[April Fools] Votekick #74392

Closed
wants to merge 4 commits into from
Closed

Conversation

RigglePrime
Copy link
Member

Pretty self-explanatory

@Sealed101 Sealed101 added the April Fools This is going to ruin the upcoming week label Mar 31, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Mar 31, 2023

This pull request was test merged in 76 round(s).

Round list

sybil

manuel

terry

to_chat(world, span_boldannounce("Vote kicking [display_name] failed, there weren't enough votes! Kicking [initiator], the vote creator instead."))
if(initiator)
message_admins("VOTEKICK: [initiator] got kicked for failing a kick vote.[initiator.holder ? " Being an admin doesn't save you from fate." : ""]")
to_chat(initiator, span_danger("You have been kicked from the server instead of your target! How tragic."), confidential = TRUE)
Copy link
Member

Choose a reason for hiding this comment

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

you have to use to_chat_immeadate if you want the target to see this during lag

to_chat(world, span_boldannounce("Vote kicking [display_name] succeeded!"))

if(!lucky_contender.holder)
to_chat(lucky_contender, span_danger("You have been vote kicked from the server!"), confidential = TRUE)
Copy link
Member

Choose a reason for hiding this comment

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

you have to use to_chat_immeadate if you want the target to see this during lag

@Mothblocks
Copy link
Member

For the benefit of the maintainers, April Fools PRs will be labeled and closed. Key Holders are still able to test merge PRs that are closed, so this makes no difference to you, the author, assuming you were able to find a Key Holder to merge this PR in the first place. If you need to make updates to this PR, you can simply add to the branch, and when the TM updates it will use those changes, even if they do not show up to the PR.

@Mothblocks Mothblocks closed this Mar 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
April Fools This is going to ruin the upcoming week
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants