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

[Port] Autovote System #62

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

PuroSlavKing
Copy link

Описание PR

Taken from: Simple-Station/Einstein-Engines#1213

Система автоматических голосований. После конца раунда автоматически запускаются голосования за следующий режим и карту.

Почему / Баланс

Педалям меньше кнопочек жать

Ссылка на ветку

Технические детали

Медиа

image

Критические изменения

Список изменений
🆑 sleepyyapril

  • add: Добавлена система автоматических голосований. После конца раунда автоматически запускаются голосования за следующий режим и карту.

Co-Authored-By: sleepyyapril <[email protected]>
@FireNameFN
Copy link
Collaborator

На Корваксе, вроде, отключены голосования от игроков за карты.

@PuroSlavKing
Copy link
Author

На Корваксе, вроде, отключены голосования от игроков за карты.

Оно и так выключено, нужно отдельно в конфиге каждого сервера включить

@AwareFoxy
Copy link
Collaborator

Пойду у Ксено спрошу

Copy link
Collaborator

@Tornado-Technology Tornado-Technology left a comment

Choose a reason for hiding this comment

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

Исты сделали довольно локаничный код, есть стилистические приебки, но каждый пишет как хочет

@lzk228
Copy link
Collaborator

lzk228 commented Nov 18, 2024

БЛЯ НУ НЕ НАДО ЭТО, РАНДОМ КАРТЫ ЭТО ТОП НАМ НЕ НАДО ГОЛОСОВАНИЯ

@PuroSlavKing
Copy link
Author

БЛЯ НУ НЕ НАДО ЭТО, РАНДОМ КАРТЫ ЭТО ТОП НАМ НЕ НАДО ГОЛОСОВАНИЯ

Боится, что голоса за силли айланд уйдут в минус 😈

@NotSoDana
Copy link

Педалям меньше кнопочек жать

Педалям запрещены голосования за карты..
За режим ещё ок, но за карту резко против, ибо начнётся ебаный дроч дельты (опять)

@PuroSlavKing
Copy link
Author

Педалям меньше кнопочек жать

Педалям запрещены голосования за карты..
За режим ещё ок, но за карту резко против, ибо начнётся ебаный дроч дельты (опять)

Кинешь ПР выключающий голосования за карты на твоём сервере

@NotSoDana
Copy link

NotSoDana commented Nov 20, 2024

Кинешь ПР выключающий голосования за карты на твоём сервере

Они и так выключены ВЕЗДЕ и запрещёны педалям кодексом. Зачем давать возможность снова пикать каждый раунд дельту, если игроки сами проголосовали за то чтоб ротация была рандомная. Это тупо ломает все усилия на протяжении нескольких месяцев

@Ko4ergaPunk
Copy link

Рандомная ротация и всё. Нахуя по прежнему актуальную проблему выводить на некст лвл

@PuroSlavKing
Copy link
Author

Кинешь ПР выключающий голосования за карты на твоём сервере

Они и так выключены ВЕЗДЕ и запрещёны педалям кодексом. Зачем давать возможность снова пикать каждый раунд дельту, если игроки сами проголосовали за то чтоб ротация была рандомная. Это тупо ломает все усилия на протяжении нескольких месяцев

Раз выключены, то не включайте, лол, это билд не только для корвакса

Copy link

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot added the Merge Conflict МерджКонфликт label Nov 22, 2024
@github-actions github-actions bot added S: Merge Conflict У ПРа есть мердж конфликт and removed Merge Conflict МерджКонфликт labels Nov 22, 2024
Copy link

This pull request has conflicts, please resolve those before we can evaluate the pull request.

Copy link
Collaborator

@Tornado-Technology Tornado-Technology left a comment

Choose a reason for hiding this comment

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

UwU

@Tornado-Technology
Copy link
Collaborator

Я откатил изменение МРП конфига, так что как механ, можно спокойно мерджить

@github-actions github-actions bot added size/M and removed S: Merge Conflict У ПРа есть мердж конфликт size/L labels Nov 29, 2024
@Tornado-Technology
Copy link
Collaborator

@PuroSlavKing на RMC есть система, что бы 1 и ту же карту нельзя было пикать X раундов после её пика, что бы не дрочить одну карт

@AwareFoxy
Copy link
Collaborator

Сделать бы это возможность выключать кнопочкой

/// <summary>
/// CorvaxNext Surgery cvars
/// _CorvaxNext Surgery cvars
Copy link
Collaborator

Choose a reason for hiding this comment

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

Лучше оставь.

Comment on lines +23 to +41
/// <summary>
/// Enables the automatic voting system.
/// <summary>
public static readonly CVarDef<bool> AutoVoteEnabled =
CVarDef.Create("vote.autovote_enabled", false, CVar.SERVERONLY);

/// <summary>
/// Automatically starts a map vote when returning to the lobby.
/// Requires auto voting to be enabled.
/// <summary>
public static readonly CVarDef<bool> MapAutoVoteEnabled =
CVarDef.Create("vote.map_autovote_enabled", true, CVar.SERVERONLY);

/// <summary>
/// Automatically starts a gamemode vote when returning to the lobby.
/// Requires auto voting to be enabled.
/// <summary>
public static readonly CVarDef<bool> PresetAutoVoteEnabled =
CVarDef.Create("vote.preset_autovote_enabled", true, CVar.SERVERONLY);
Copy link
Collaborator

Choose a reason for hiding this comment

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

Отступа слишком много, кажется.


public override void Initialize()
{
base.Initialize();
Copy link
Collaborator

Choose a reason for hiding this comment

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

Вызов пустого метода.

Copy link
Collaborator

Choose a reason for hiding this comment

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

base должен вызывается всегда, считается хорошим тоном

SubscribeLocalEvent<PlayerJoinedLobbyEvent>(OnPlayerJoinedLobby);
}

public void OnReturnedToLobby(RoundRestartCleanupEvent ev) => CallAutovote();
Copy link
Collaborator

Choose a reason for hiding this comment

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

Ванлайнеры делают только на свойствах.

Copy link

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot added the S: Merge Conflict У ПРа есть мердж конфликт label Dec 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S: Merge Conflict У ПРа есть мердж конфликт S: Needs Review ПРу необходим ревью size/M Status: Needs Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants