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.

Sign up for GitHub

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

Jump to bottom

Nukeops Gamma #2486

Merged
merged 8 commits into from
Feb 28, 2025
Merged

Nukeops Gamma #2486

merged 8 commits into from
Feb 28, 2025
+83 −1

Conversation

canvaswalker
Copy link

@canvaswalker canvaswalker commented Jan 14, 2025 •
edited
Loading

Описание PR

Теперь после объявление войны Ядерными Оперативниками автоматически ставится Гамма код, отключаются станционные факсы, а так же ивенты

ЭТОТ АПДЕЙТ НЕ ЗАТРАГИВАЕТ АДМИН КНОПКИ, ОНИ ВСЁ ЕЩЁ МОГУТ ЗАПУШИТЬ ЛЮБОЙ ИВЕНТ САМОСТОЯТЕЛЬНО!
Медиа

Проверки

  • PR полностью завершён и мне не нужна помощь чтобы его закончить.
  • Я ознакомился с наставлениями по работе с репозиторием и следовал им при создании PR'а.
  • Я внимательно просмотрел все свои изменения и багов в них не нашёл.
  • Я запускал локальный сервер со своими изменениями и всё протестировал.
  • Я добавил скриншот/видео демонстрации PR в игре, или этот PR этого не требует.

Изменения

🆑 Gnomeev Dynamics

  • add: Теперь после объявление войны Ядерными Оперативниками автоматически ставится Гамма код, отключаются станционные факсы, а так же ивенты!

@github-actions github-actions bot added the Changes: C# Изменение затронуло файлы ".cs" label Jan 14, 2025
@canvaswalker
Copy link
Author

@UrPrice

@canvaswalker
Copy link
Author

Такое уже есть у жнеца и культа, если не ошибаюсь. Так-что пора бы и дать нюкопсу, на который по умолчанию не работают ВХ и ПГ :3

@TryHardo7
Copy link

TryHardo7 commented Jan 14, 2025 •
edited
Loading

Мы специально не ставим гамму, потому что станция вызвает эвак, а при гамме он прилетает через 5 минут.
Это дурацкая ситуация при которой приходится спамить озывом, либо через АХ вмешиваться и говорить перестать, либо эвак прилетает прямо при нюке на станции. Такое было неприятное количество раз.
И нет, консоль которая стоит у нюки с отзывом не помогает, потому что дежурить у неё никому не интересный геймплей, не говоря уже о том, что вечный спам алёртами вызова и отзыва эвака раздражает абсолютно всех.
Либо уже ограничить кодом возможность вызова эвака пока нюка жива, или до определённого времени после объявы войны, либо не трогать и дать админам и дальше с этим разбираться.

@anders0n9
Copy link

Мы специально не ставим гамму, потому что станция вызвает эвак, а при гамме он прилетает через 5 минут. Это дурацкая ситуация при которой приходится спамить озывом, либо через АХ вмешиваться и говорить перестать, либо эвак прилетает прямо при нюке на станции. Такое было неприятное количество раз. И нет, консоль которая стоит у нюки с отзывом не помогает, потому что дежурить у неё никому не интересный геймплей, не говоря уже о том, что вечный спам алёртами вызова и отзыва эвака раздражает абсолютно всех. Либо уже ограничить кодом возможность вызова эвака пока нюка жива, или до определённого времени после объявы войны, либо не трогать и дать админам и дальше с этим разбираться.

Эвак не призывается в первые 15 минут вроде, или больше. Дальше сугубо IC из-за промедления нюкеров.

@stalengd stalengd added the Need to be discussed Требуется обсуждение перед аппрувом/мёрджем label Jan 15, 2025
@UrPrice
Copy link
Collaborator

UrPrice commented Jan 15, 2025

Для этого нужно вот это:
{D60F5205-B9BC-4A0F-BEA9-DDAACB06B333}

@canvaswalker
Copy link
Author

Для этого нужно вот это: {D60F5205-B9BC-4A0F-BEA9-DDAACB06B333}

окей, сделаю

@Kirus59
Copy link

Kirus59 commented Jan 17, 2025

бля, не туда

@UrPrice
Copy link
Collaborator

UrPrice commented Jan 17, 2025

В Ramp и Basic планировщики поступает таблица BasicGameRulesTable, которая включает в себя множество неуместных в условиях отключенной связи и войны.

В NukeOps используется BasicStationEventScheduler. При условиях объявления войны, желательно, чтобы этот планировщик исключался из геймпресета.

@FriedrichStrassen
Copy link

А можно вы не будете ограничивать алёрты ЦК? Админские кнопки не трож

@UrPrice
Copy link
Collaborator

UrPrice commented Feb 7, 2025

А можно вы не будете ограничивать алёрты ЦК? Админские кнопки не трож

Причём тут вы вообще?

@UrPrice
Copy link
Collaborator

UrPrice commented Feb 7, 2025

Не знаю что ВМТ имел в виду. Разбираться в трёхтонной беседе ещё более впадлу, если сразу не могут написать по делу.

@Kirus59 по факту ревью - проверишь, чтобы наложенные ограничения не мешали кон. командам админов, заранее спасибо.

@Kirus59
Copy link

Kirus59 commented Feb 7, 2025

Пока в драфт закину, до этих изменений #2486 (comment)

@Kirus59 Kirus59 marked this pull request as draft February 7, 2025 09:39
@canvaswalker canvaswalker marked this pull request as ready for review February 27, 2025 15:26
@canvaswalker
Copy link
Author

@Kirus59
I need a review review, a review is what I need
Hey, hey
Well I need a review review a review that's what I need

UrPrice
UrPrice approved these changes Feb 27, 2025
View reviewed changes
Copy link
Collaborator

@UrPrice UrPrice left a comment

Choose a reason for hiding this comment

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

clap clap clap

UrPrice
UrPrice requested changes Feb 27, 2025
View reviewed changes
Copy link
Collaborator

@UrPrice UrPrice left a comment

Choose a reason for hiding this comment

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

Поторопился.
Из под агоста возможность вызова эвака должна оставаться.
{F8267061-4B84-4916-8820-A2BB3870EDD0}

@canvaswalker
Copy link
Author

Поторопился. Из под агоста возможность вызова эвака должна оставаться. {F8267061-4B84-4916-8820-A2BB3870EDD0}

Поторопился. Из под агоста возможность вызова эвака должна оставаться. {F8267061-4B84-4916-8820-A2BB3870EDD0}

Это вообще оффовские штуки btw. Нужно именно через агост? В админ меню есть же кнопка на эвак

@UrPrice
Copy link
Collaborator

UrPrice commented Feb 27, 2025

Это вообще оффовские штуки btw. Нужно именно через агост? В админ меню есть же кнопка на эвак

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

@UrPrice
Copy link
Collaborator

UrPrice commented Feb 27, 2025

В админ меню есть же кнопка на эвак

Да, проверил работают. Тогда ок всё.

@UrPrice UrPrice requested a review from Kirus59 February 27, 2025 19:34
@Kirus59
Copy link

Kirus59 commented Feb 28, 2025

@UrPrice факсы должны в целом не работать или только Станция -> ЦК?
Также разве должны отключаться все мидраунд ивенты после объявления?

@UrPrice
Copy link
Collaborator

UrPrice commented Feb 28, 2025

@UrPrice факсы должны в целом не работать или только Станция -> ЦК? Также разве должны отключаться все мидраунд ивенты после объявления?

варопс - это пвп станция против нюки, так что мидраунд ивенты лишние там ни к чему
подойдёт и тот вариант что есть, но если получится ограничить только по станция -> ЦК, конечно будет получше. В любом случае админ через F7 - админ факс - может отправить нормальный факс

@Kirus59
Copy link

Kirus59 commented Feb 28, 2025

варопс - это пвп станция против нюки, так что мидраунд ивенты лишние там ни к чему

Мб тогда из пресета лучше убрать, или они не должны быть именно после объявы?

@canvaswalker
Copy link
Author

по рэпэ связь выключается, нету ивентов
мне кажется легит

@canvaswalker
Copy link
Author

плюс, если убрать из присета стелс нюкопс без объявы станет еще более явным и игроки начнут его палить по отсутствию ивентов в мидгейме

@UrPrice
Copy link
Collaborator

UrPrice commented Feb 28, 2025

варопс - это пвп станция против нюки, так что мидраунд ивенты лишние там ни к чему

Мб тогда из пресета лучше убрать, или они не должны быть именно после объявы?

не должны быть именно после объявы

@Kirus59
Copy link

Kirus59 commented Feb 28, 2025

по рэпэ связь выключается

Я согласен в контексте внешней связи, но как они могут повлиять на внутреннюю (факсы Станция -> Станция)?
Хоть факсами после объявы навряд ли будут пользоваться, но все же

Kirus59
Kirus59 requested changes Feb 28, 2025
View reviewed changes
Copy link

@Kirus59 Kirus59 left a comment

Choose a reason for hiding this comment

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

В остальном норм, проверку на запрет сообщения только на внестанционные факсы добавлять будешь?

Content.Server/Fax/FaxSystem.cs Outdated

TryComp<NameModifierComponent>(sendEntity, out var nameMod);
Copy link

Choose a reason for hiding this comment

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

Тут пометку забыл

Content.Shared/Fax/Components/FaxMachineComponent.cs Outdated

//ss220 autogamma update
[ByRefEvent]
public record struct FaxSendAttemptEvent(EntityUid FaxEnt, string DestinationFaxAddress, string SenderFaxAddress)
Copy link

Choose a reason for hiding this comment

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

Унаследуй от CancellableEntityEventArgs

@UrPrice UrPrice removed Need to be discussed Требуется обсуждение перед аппрувом/мёрджем Status: Needs Review labels Feb 28, 2025
@canvaswalker canvaswalker requested a review from Kirus59 February 28, 2025 18:01
@Kirus59 Kirus59 merged commit f0e8acb into SerbiaStrong-220:master Feb 28, 2025
12 checks passed
@canvaswalker canvaswalker deleted the gamma-my-beloved branch February 28, 2025 20:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@UrPrice UrPrice UrPrice approved these changes

@Kirus59 Kirus59 Kirus59 approved these changes

Assignees

@Kirus59 Kirus59

Labels
Changes: C# Изменение затронуло файлы ".cs"
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
8 participants
@canvaswalker @TryHardo7 @anders0n9 @UrPrice @Kirus59 @FriedrichStrassen @stalengd @Ady4ik