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

Update basevotes #1249

Merged
merged 2 commits into from
Jun 13, 2020
Merged

Update basevotes #1249

merged 2 commits into from
Jun 13, 2020

Conversation

accelerator74
Copy link
Contributor

When a player leaves during a voteban, he will be banned anyway. Also added a cvar with a ban time setting.

When a player leaves during a voteban, he will be banned anyway. Also added a cvar with a ban time setting.
@crashzk
Copy link
Contributor

crashzk commented Apr 26, 2020

This could be done in the Mute, Gag & Silence polls. I believe you must have the same problem right? If the player leaves the game during the vote, he is not punished.

@asherkin
Copy link
Member

This could be done in the Mute, Gag & Silence polls. I believe you must have the same problem right? If the player leaves the game during the vote, he is not punished.

A player could re-join after any of those to have the effect removed anyway - that is where voteban is different.

Copy link
Member

@Headline Headline left a comment

Choose a reason for hiding this comment

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

Just one small comment inline

Has this been tested?

plugins/basevotes.sp Outdated Show resolved Hide resolved
@Headline Headline added the Feature Request user requested feature label May 8, 2020
@accelerator74
Copy link
Contributor Author

Just one small comment inline

Has this been tested?

For me it works at normally :)

Copy link
Member

@Headline Headline left a comment

Choose a reason for hiding this comment

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

Thanks!

@Headline Headline merged commit 2d971a9 into alliedmodders:master Jun 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Request user requested feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants