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

Добавляет трейторам новую цель на взрыв отсека #12909

Merged
merged 10 commits into from
Apr 8, 2024

Conversation

simb11
Copy link
Contributor

@simb11 simb11 commented Mar 3, 2024

Описание изменений

Добавляет предателям цель на подрыв какого-нибудь важного отсека
Трейтор при получении цели получает ещё и маяк для призыва бомбы.
Собственно, цель трейтора заключается в том чтобы пробраться в нужный отсек, установить бомбу и дождаться взрыва.
Удалил аблятивную броню из списка предметов для кражи.
И ещё переделал код синдибомбы чтобы было меньше багов и было красивее

Почему и что этот ПР улучшит

Больше интересных целей меньше неинтересных

Авторство

Чеинжлог

🆑 Simbaka

  • add: Предателю добавлено новое задание: Взорвать один из отсеков станции.
  • del: Предателям больше не выпадает цель на кражу аблятивной брони.

@TauKitty
Copy link
Contributor

TauKitty commented Mar 3, 2024

Changelog status: ✔️

@TauKitty TauKitty added Feature Новая фича Revert / Removal labels Mar 3, 2024
Copy link
Contributor

@maleyvich maleyvich left a comment

Choose a reason for hiding this comment

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

если внезапно пр на перевод целек замержат быстрее чем этот (в чём я конечно сомневаюсь), можешь либо свой перевод сделать либо взять этот

code/game/gamemodes/objectives/bomb.dm Show resolved Hide resolved
code/game/machinery/syndie_big_bomb.dm Outdated Show resolved Hide resolved
code/game/machinery/syndie_big_bomb.dm Outdated Show resolved Hide resolved
code/game/machinery/syndie_big_bomb.dm Outdated Show resolved Hide resolved
code/game/machinery/syndie_big_bomb.dm Show resolved Hide resolved
code/game/machinery/syndie_big_bomb.dm Outdated Show resolved Hide resolved
code/game/machinery/syndie_big_bomb.dm Show resolved Hide resolved
@simb11
Copy link
Contributor Author

simb11 commented Mar 3, 2024

ладно

@Pufikc
Copy link
Contributor

Pufikc commented Mar 3, 2024

Хм.. бомбу то возможно обезвредить? Пока оно тикает там, пока кто-то заметит из персонала и сообщит СБ. Может появится ещё больше смысла сапёрству для СБ?

@simb11
Copy link
Contributor Author

simb11 commented Mar 3, 2024

всегда можно было

@muWander
Copy link
Contributor

muWander commented Mar 3, 2024

было бы оч круто если бы отсек рандомился из списка но если тебе лень то не надо

Copy link
Contributor

@AzzyDreemurr13 AzzyDreemurr13 left a comment

Choose a reason for hiding this comment

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

Вставлю свои 5 пердеповских копеек против. Ибо у нас уже есть отревьювленный ПР #12843 с переводом целек от переводчиков уже 2 недели. Осталось повторная проверка от мейнтеров и у тебя здесь всего лишь 8 строк текста. Крайне глупо будет выглядеть, ещё один слой английского текста с готовым переводом целек.

@simb11
Copy link
Contributor Author

simb11 commented Mar 3, 2024

ладн перевёл

Copy link
Contributor

@AzzyDreemurr13 AzzyDreemurr13 left a comment

Choose a reason for hiding this comment

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

Ладно, хотя бы целька. Убираю свои 5 копеек :dovolen:
(Спасибо за понимание)

code/game/gamemodes/objectives/bomb.dm Show resolved Hide resolved
code/game/gamemodes/objectives/bomb.dm Show resolved Hide resolved
code/game/machinery/syndie_big_bomb.dm Outdated Show resolved Hide resolved
code/game/machinery/syndie_big_bomb.dm Outdated Show resolved Hide resolved
code/game/machinery/syndie_big_bomb.dm Outdated Show resolved Hide resolved
code/game/machinery/syndie_big_bomb.dm Outdated Show resolved Hide resolved
code/game/machinery/syndie_big_bomb.dm Outdated Show resolved Hide resolved
@@ -43,8 +41,6 @@ ADD_TO_POIFS_LIST(/obj/item/stack/sheet/mineral/uranium)
"the hypospray" = /obj/item/weapon/reagent_containers/hypospray/cmo,
"a Research Director's teleport armor" = /obj/item/clothing/suit/armor/vest/reactive,
"the captain's pinpointer" = /obj/item/weapon/pinpointer,
"an ablative armor vest" = /obj/item/clothing/suit/armor/laserproof,
"the golden fire extinguisher" = /obj/item/weapon/reagent_containers/spray/extinguisher/golden,
Copy link
Contributor

Choose a reason for hiding this comment

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

А я против удаления цели на петушитель.

Copy link
Contributor

Choose a reason for hiding this comment

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

Ну золотой хоть оставь. Хуй с тем аблятивом

Copy link
Contributor

@NinjaPikachuska NinjaPikachuska left a comment

Choose a reason for hiding this comment

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

нужно на сигналы переделать
https://github.com/TauCetiStation/TauCetiClassic/pull/12909/files#r1518588848

@NinjaPikachuska
Copy link
Contributor

еще очень спорный выпил золотого огнетушителя из целей
было бы хорошо проблему его хоть описать

@simb11
Copy link
Contributor Author

simb11 commented Mar 22, 2024

еще очень спорный выпил золотого огнетушителя из целей

Мне не нравится то что он никому не нужен (кроме визарда), никому не принадлежит и от его кражи никто ничего не теряет

@Slavik2001
Copy link
Contributor

Как не нужен? У него урон и за того же тритора разливать воду

@simb11
Copy link
Contributor Author

simb11 commented Mar 22, 2024

никто кроме визорда не использует его как оружие
а мочить полы можно (что и делают) и обычным огнетушителем

@volas
Copy link
Member

volas commented Mar 23, 2024

и от его кражи никто ничего не теряет

станция теряет в робастности и престиже, это как знамя вражеской армии украсть

Copy link
Member

@volas volas left a comment

Choose a reason for hiding this comment

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

По коду больше проблем не имею, но не очень с идеей удаления цели на огнетушитель. Какое-то хранилище тогда вообще никому не нужное получается, даже триторам.

@simb11
Copy link
Contributor Author

simb11 commented Mar 24, 2024

@volas впилил обратно цель на огнетушитель

Copy link

github-actions bot commented Apr 7, 2024

Данный ПР автоматически отмечен как застоявшийся по причине длительного отсутствия обновлений. Он будет закрыт через 7 дней, если никакой активности не будет проявлено. Если вы считаете, что ПР еще актуален, или что я (злобный робот) пристаю к вам зря - просто напишите любой комментарий. Спасибо за ваш вклад.

@volas volas changed the title Добавляет трейторам новую цель2 Добавляет трейторам новую цель на взрыв отсека Apr 8, 2024
@volas volas merged commit 0fb483c into TauCetiStation:master Apr 8, 2024
12 checks passed
TauKitty added a commit to TauCetiStation/ClassicChangelog that referenced this pull request Apr 8, 2024
@volas
Copy link
Member

volas commented Apr 8, 2024

наконец-то легальный повод взрывать станцию

@simb11 simb11 deleted the new-cel branch May 7, 2024 16:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.