Spacelaw Change Rule #97

AwareFoxy · 2024-11-29T23:20:40Z

Автор сия творения - @chavonadelal

Описание PR

Теперь может выпасть ОЧЕНЬ РЕДКИЙ ивент на смену корп закона xdd:

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

SS13 content

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

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

@Tornado-Technology удачи

Медиа

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

Список изменений
Для чейнджлогера - в качестве автора укажи to4no_fix в ДС
🆑

add: Добавлен очень редкий ивент с возможностью смены корпоративных законов!

CaptainSqrBeard · 2024-11-29T23:31:12Z

это уже можно тестировать?

AwareFoxy · 2024-11-29T23:57:51Z

это уже можно тестировать?

там нечего тестировать

lzk228

в описании ПРа опиши как собственно происходит этот ивент либо добавь медиа

CaptainSqrBeard · 2024-11-30T06:34:50Z

это уже можно тестировать?

там нечего тестировать

ЛОЖЬ

Vonsant

Лрп бред сивой кобылы под метамфетамином. Для мрп большая часть "законов" не подходит. Либо переписать в более серьезном виде, либо принято не будет.

Resources/Locale/ru-RU/_CorvaxNext/station-events/events/space-law-change.ftl

Vonsant · 2024-11-30T07:18:01Z

Если делаете лрп контент, то для него нужен отдельный цварник, чтобы можно было включать его в серверных настройках. Но по умолчанию должно быть выключено.

Условно rp_lrp = false.

Подавляющее число серверов корвакса позиционируется как мрп, где подобные шутки хуютки с пресмыканием перед клоуном будут восприняты в штыки.

chavonadelal

Убрать варианты, связанные с оружием, операциями, лекарствами и про танцевание с клоуном, потому что от них, возможно, действительно могут быть проблемы. Варианты связанные с оружием, операциями и лекарствами вообще не мои, хех. Остальные варианты забавные, с учетом того, что ивент может выпасть всего один раз за раунд с небольшим шансом. Добавьте веселья игрокам, работки им и так хватает. Фокси опередил меня же и зачем-то черепикнул мою ветку в форке 🤔 , так бы сам поправил

CaptainSqrBeard · 2024-11-30T10:04:59Z

Лрп бред сивой кобылы под метамфетамином. Для мрп большая часть "законов" не подходит. Либо переписать в более серьезном виде, либо принято не будет.

Помоему вся суть этого ивента в бредовых 1984 законах.

К тому-же я не очень понимаю по какому критерию ты отбираешь ЛРПшные законы, которые бы мешали на МРП. Но этот вопрос я лучше оставлю тому, кто понимает как играются МРП сервера.

Resources/Locale/ru-RU/_CorvaxNext/station-events/events/space-law-change.ftl

Vonsant · 2024-12-01T12:09:13Z

@AwareFoxy старший модератор провел обсуждение с модераторами серверов и было принято решение, что данный ивент допустим только на сервере Мейн. Так что цварник делать все таки придется.

AwareFoxy · 2024-12-01T12:10:10Z

@AwareFoxy старший модератор провел обсуждение с модераторами серверов и было принято решение, что данный ивент допустим только на сервере Мейн. Так что цварник делать все таки придется.

я это в пуссицвар впишу тхен, ок?

Vonsant · 2024-12-01T12:11:59Z

я это в пуссицвар впишу тхен, ок?

idk, главное чтобы в будущем этот цварник можно было универсально использовать под разные фичи. В идеале я бы сделала три отдельных фича тоггла под каждый уровень сервера,у словно "мрп-" "мрп" "мрп+", чтобы потом можно было распределять фичи по серверам, а не все сервера единообразными делать.

chavonadelal · 2024-12-01T12:13:19Z

@AwareFoxy старший модератор провел обсуждение с модераторами серверов и было принято решение, что данный ивент допустим только на сервере Мейн. Так что цварник делать все таки придется.

Вот это вы скучные

chavonadelal · 2024-12-01T12:13:54Z

@AwareFoxy раз это для лрп, то верни" ЛРП Бред" законы

Vonsant · 2024-12-01T12:14:28Z

@AwareFoxy раз это для лрп, то верни" ЛРП Бред" законы

Не, их точно запретили даже для мейна.

Vonsant · 2024-12-01T12:14:55Z

Потому что мейн не лрп, а мрп сервер с пониженными требованиями.

Vonsant · 2024-12-01T12:19:29Z

Потом если людям понравится, то будет переключено и на другие сервера, благо с цварниками просто работать.

chavonadelal · 2024-12-01T12:27:25Z

Потом если людям понравится, то будет переключено и на другие сервера, благо с цварниками просто работать.

там 0 онлайна, некому нравится будет

chavonadelal · 2024-12-01T12:27:59Z

при том, что ивент с 50 онлайна только может выпасть, с меньшим онлайном в нем нет смысла. глупо в целом отрубать ивент, когда можно просто сделать разные датасеты для разных серверов и выбирать какой использовать через цвариник
lawMRP-LocalizedDataset
lawMRPLocalizedDataset
lawMRP+LocalizedDataset

Content.Server/_CorvaxNext/StationEvents/Events/SpaceLawChange.cs

Co-authored-by: FN <[email protected]>

lzk228

ревью кода только потому что я не хочу, чтобы вы мёржили какое-то говно
я не хочу чтобы это было мёржнуто вовсе, но МАКСИМАЛЬНО не хочу, пока:

не будет добавлен способ для администрации и присоединившихся позже игроков узнать об изменённых законах СРАЗУ как только они присоединяются в раунд, я не буду предлагать как это делать, скажу лишь что это ОБЯЗАННО быть перед добавлением ивента в игру

lzk228 · 2024-12-09T22:32:31Z

Content.Shared/_CorvaxNext/NextVars.cs

+    /// <summary>
+    /// LRP CVar
+    /// </summary>
+    public static readonly CVarDef<bool> LRPEnabled =
+        CVarDef.Create("lrp.enabled", false, CVar.SERVERONLY);


лучше сделать отдельный цвар для этого ивента, а не общий "лрп"

lzk228 · 2024-12-09T22:36:30Z

Content.Server/_CorvaxNext/StationEvents/Events/SpaceLawChange.cs

+            if (!_cfg.GetCVar(NextVars.LRPEnabled))
+                return;


а оно не сделает обьявление ивента, после которого ничего не произойдёт если конфиг отключен?

lzk228 · 2024-12-09T22:44:55Z

Resources/Locale/ru-RU/_CorvaxNext/station-events/events/space-law-change.ftl

+    Продолжительность смены на момент отправки факса: { $stationTime }
+    { station-event-space-law-change-fax-end }
+
+station-event-space-law-change-announcement = В связи с последними изменениями в корпоративной политике, { $essence } теперь признается (или признаются) незаконным(-ыми) по кодовому номеру «XX1» Корпоративного закона. В ваших интересах незамедлительно исправить ситуацию до вступления изменений в силу через { $time } минут. Служба безопасности и Командование обязаны приложить все возможные усилия для обеспечения полного соответствия станции требованиям закона к моменту окончания периода адаптации.


Suggested change

station-event-space-law-change-announcement = В связи с последними изменениями в корпоративной политике, { $essence } теперь признается (или признаются) незаконным(-ыми) по кодовому номеру «XX1» Корпоративного закона. В ваших интересах незамедлительно исправить ситуацию до вступления изменений в силу через { $time } минут. Служба безопасности и Командование обязаны приложить все возможные усилия для обеспечения полного соответствия станции требованиям закона к моменту окончания периода адаптации.

station-event-space-law-change-announcement = В связи с последними изменениями в корпоративной политике, { $essence } теперь признаётся (или признаются) незаконным(-и) по кодовому номеру «XX1» Корпоративного закона. В ваших интересах незамедлительно исправить ситуацию до вступления изменений в силу через { $time } минут. Служба безопасности и Командование обязаны приложить все возможные усилия для обеспечения полного соответствия станции требованиям закона к моменту окончания периода адаптации.

lzk228 · 2024-12-09T22:45:39Z

Content.Server/_CorvaxNext/StationEvents/Components/SpaceLawChangeRuleComponent.cs

+        public ProtoId<LocalizedDatasetPrototype> LawLocalizedDataset { get; set; }
+
+        /// <summary>
+        /// Time before changes to the law come into force.


Suggested change

/// Time before changes to the law come into force.

/// Time in minutes before changes to the law come into force.

lzk228 · 2024-12-09T22:55:13Z

Content.Server/_CorvaxNext/StationEvents/Events/SpaceLawChange.cs

+                Loc.GetString("materials-paper"),
+                null,
+                null,
+                "paper_stamp-centcom",
+                new List<StampDisplayInfo>


Suggested change

Loc.GetString("materials-paper"),

null,

null,

"paper_stamp-centcom",

new List<StampDisplayInfo>

Loc.GetString("materials-paper"),

stampState: "paper_stamp-centcom",

stampedBy: new List<StampDisplayInfo>

по сути должно работать

lzk228 · 2024-12-09T22:57:44Z

Resources/Locale/ru-RU/_CorvaxNext/station-events/events/space-law-change.ftl

+station-event-space-law-change-essence-8 = деятельность руководителей отделов, которые не уделяют должного внимания патриотическому воспитанию своих сотрудников во славу NanoTrasen
+station-event-space-law-change-essence-9 = не ношение головного убора
+station-event-space-law-change-essence-10 = неисполнение главой отдела личного приветствия каждого члена экипажа, посещающего его отдел


удалить нахуй

lzk228 · 2024-12-09T22:58:22Z

Resources/Locale/ru-RU/_CorvaxNext/station-events/events/space-law-change.ftl

+station-event-space-law-change-essence-14 = общение не шепотом вне каналов связи
+station-event-space-law-change-essence-15 = попытки убедить Капитана станции в существовании мировых заговоров
+station-event-space-law-change-essence-16 = введение в оборот фруктов и овощей нового урожая без предварительной дегустации каждого вида Главой персонала
+station-event-space-law-change-essence-17 = неисполнение приветствия "Привет, товарищи" при входе в Бриг


github-actions · 2024-12-17T15:24:02Z

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

chavonadelal and others added 2 commits November 30, 2024 00:12

Adding a new SpaceLawChange station event

63bd861

добавил чут-чут новых законов

247af59

AwareFoxy requested review from Vonsant, Tornado-Technology and FireNameFN as code owners November 29, 2024 23:20

github-actions bot added S: Needs Review ПРу необходим ревью S: Untriaged Changes: Localization Изменяет локализацию size/L labels Nov 29, 2024

noSpawn >>> abstract

0720f8a

lzk228 requested changes Nov 30, 2024

View reviewed changes

Vonsant requested changes Nov 30, 2024

View reviewed changes

chavonadelal suggested changes Nov 30, 2024

View reviewed changes

CaptainSqrBeard reviewed Nov 30, 2024

View reviewed changes

Resources/Locale/ru-RU/_CorvaxNext/station-events/events/space-law-change.ftl Outdated Show resolved Hide resolved

remove lrp laws

3032c9b

AwareFoxy requested a review from Vonsant December 1, 2024 12:02

лрп цварник

d9d2e34

FireNameFN requested changes Dec 4, 2024

View reviewed changes

Apply suggestions from code review

0223e54

Co-authored-by: FN <[email protected]>

lzk228 requested changes Dec 9, 2024

View reviewed changes

lzk228 added the Status: Awaiting Changes label Dec 9, 2024

github-actions bot added the S: Merge Conflict У ПРа есть мердж конфликт label Dec 17, 2024

AwareFoxy closed this Dec 18, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Spacelaw Change Rule #97

Spacelaw Change Rule #97

AwareFoxy commented Nov 29, 2024

CaptainSqrBeard commented Nov 29, 2024

AwareFoxy commented Nov 29, 2024

lzk228 left a comment

CaptainSqrBeard commented Nov 30, 2024

Vonsant left a comment

Vonsant commented Nov 30, 2024

chavonadelal left a comment •

edited

Loading

CaptainSqrBeard commented Nov 30, 2024

Vonsant commented Dec 1, 2024

AwareFoxy commented Dec 1, 2024

Vonsant commented Dec 1, 2024

chavonadelal commented Dec 1, 2024

chavonadelal commented Dec 1, 2024

Vonsant commented Dec 1, 2024

Vonsant commented Dec 1, 2024

Vonsant commented Dec 1, 2024

chavonadelal commented Dec 1, 2024

chavonadelal commented Dec 1, 2024 •

edited

Loading

lzk228 left a comment

lzk228 Dec 9, 2024

lzk228 Dec 9, 2024

lzk228 Dec 9, 2024

lzk228 Dec 9, 2024

lzk228 Dec 9, 2024

lzk228 Dec 9, 2024

lzk228 Dec 9, 2024

github-actions bot commented Dec 17, 2024

	/// Time before changes to the law come into force.
	/// Time in minutes before changes to the law come into force.

Spacelaw Change Rule #97

Spacelaw Change Rule #97

Conversation

AwareFoxy commented Nov 29, 2024

Описание PR

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

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

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

Медиа

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

CaptainSqrBeard commented Nov 29, 2024

AwareFoxy commented Nov 29, 2024

lzk228 left a comment

Choose a reason for hiding this comment

CaptainSqrBeard commented Nov 30, 2024

Vonsant left a comment

Choose a reason for hiding this comment

Vonsant commented Nov 30, 2024

chavonadelal left a comment • edited Loading

Choose a reason for hiding this comment

CaptainSqrBeard commented Nov 30, 2024

Vonsant commented Dec 1, 2024

AwareFoxy commented Dec 1, 2024

Vonsant commented Dec 1, 2024

chavonadelal commented Dec 1, 2024

chavonadelal commented Dec 1, 2024

Vonsant commented Dec 1, 2024

Vonsant commented Dec 1, 2024

Vonsant commented Dec 1, 2024

chavonadelal commented Dec 1, 2024

chavonadelal commented Dec 1, 2024 • edited Loading

lzk228 left a comment

Choose a reason for hiding this comment

lzk228 Dec 9, 2024

Choose a reason for hiding this comment

lzk228 Dec 9, 2024

Choose a reason for hiding this comment

lzk228 Dec 9, 2024

Choose a reason for hiding this comment

lzk228 Dec 9, 2024

Choose a reason for hiding this comment

lzk228 Dec 9, 2024

Choose a reason for hiding this comment

lzk228 Dec 9, 2024

Choose a reason for hiding this comment

lzk228 Dec 9, 2024

Choose a reason for hiding this comment

github-actions bot commented Dec 17, 2024

chavonadelal left a comment •

edited

Loading

chavonadelal commented Dec 1, 2024 •

edited

Loading