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

Возможность болтировать техтоннели через АУК #5793

Closed
wants to merge 12 commits into from
Closed

Conversation

Motzord
Copy link
Contributor

@Motzord Motzord commented Jul 9, 2020

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

У глав в устройствах аутентификации появляются кнопки для болтирования (почти) всех техтоннелей.

Из-за криво проставленных зон некоторые входы в теха считаются зоной отсека и к сожалению не болтируютя, обещаю пофиксить как только вмержат это. (Если вмержат)

UPD.

Блокировка техтоннелей возможна лишь в код не ниже красного. Снятие блокировки доступно в любой код.

Во время радиационного шторма, со шлюзов снимаются болты и выставляется аварийный доступ

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

Больше никаких "У нас ебака в техтоннелях, не ходите там, ИИ заболтируй теха пожалуйста"

Приказ глав о запрете посещений техтоннелей, реализуем.

ИИ больше не будут заставлять тыкать по шлюзам и болтировать их.

Авторство

Motzord

Чеинжлог

🆑 Motzord

  • rscadd: В устройствах аутентификации ключ карты появилась возможность болтировать техтоннели.

@TauKitty
Copy link
Contributor

TauKitty commented Jul 9, 2020

Changelog status: ✔️

@Motzord Motzord changed the title Возможность болтировать тех через АУК Возможность болтировать техтоннели через АУК Jul 9, 2020
@AndroBetel
Copy link
Member

а чем обоснован такой жесткий нерф ролей

@Motzord
Copy link
Contributor Author

Motzord commented Jul 9, 2020

а чем обоснован такой жесткий нерф ролей

Лишь моей хотелкой, но я проведу воут на форуме.

@TauKitty
Copy link
Contributor

TauKitty commented Jul 9, 2020

This pull request has been mentioned on Tau Ceti Station Forum. There might be relevant details there:

https://forum.taucetistation.org/t/vozmozhnost-boltirovaniya-tehtonnelej-glavam/19742/1

@UDaV73rus
Copy link
Member

Тех. шлюзы в зоне отсеков и должны быть, где это не так, скорее всего упущение мапера

@cool20141

This comment was marked as abuse.

@Motzord
Copy link
Contributor Author

Motzord commented Jul 10, 2020

А если радиация?

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

@cool20141

This comment was marked as abuse.

@Motzord
Copy link
Contributor Author

Motzord commented Jul 10, 2020

Не, я имею ввиду, что во время радиации заболтируют

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

@Motzord
Copy link
Contributor Author

Motzord commented Jul 12, 2020

image
Итак, фидбек на форуме как бы намекает.

Мне уже можно просить тестмержа?

@LudwigVonChesterfield LudwigVonChesterfield added Feature Новая фича Test Merge Candidate ПР с этим лэйблом будет или уже находится с тест мерже labels Jul 15, 2020
Motzord and others added 8 commits July 25, 2020 00:03
Co-authored-by: LudwigVonChesterfield <andreydell@yahoo.com>
Co-authored-by: LudwigVonChesterfield <andreydell@yahoo.com>
Co-authored-by: LudwigVonChesterfield <andreydell@yahoo.com>
Во время радшторма нельзя болтировать теха
Я затупил, переменная не сбрасывалась после радшторма и теха нельзя было блочить
block_maintenance_off = TRUE
SSweather.run_weather("radiation storm", ZTRAIT_STATION)
block_maintenance_off = FALSE
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
block_maintenance_off = FALSE
global.block_maintenance_off = FALSE

/datum/event/radiation_storm/announce()

var/global/block_maintenance_off = FALSE
Copy link
Contributor

Choose a reason for hiding this comment

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

Но эта переменная не блокирует тех тоннели. а не позволяет блокировать их, нейминг какой-то неправильный.

А ещё, странно что переменная которая касается тех тоннелей, шлюзов, и бла-бла-бла

объявляется в файле ивента

@@ -192,3 +203,25 @@ var/global/timer_maint_revoke_id = 0
for(var/obj/machinery/door/airlock/A in M)
A.emergency = allow_state
A.update_icon()

/proc/make_bolt_all_maintenance(var/priority = FALSE)
Copy link
Contributor

Choose a reason for hiding this comment

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

Надмозговое какое-то название для функции

Зачем она вообще нужна тоже малопонятно, а то выходит что может быть аннонс о болтировании, без болтирования(если приорити = фолз)

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

@Motzord
Copy link
Contributor Author

Motzord commented Aug 14, 2020

Желание докодивать/делать красиво у меня отпало. Можно закрыть пр

@AndroBetel
Copy link
Member

ну закрой

@Motzord Motzord closed this Aug 15, 2020
@Motzord Motzord reopened this Aug 15, 2020
Motzord and others added 2 commits August 15, 2020 15:46
Co-authored-by: LudwigVonChesterfield <andreydell@yahoo.com>
Co-authored-by: LudwigVonChesterfield <andreydell@yahoo.com>
@Motzord
Copy link
Contributor Author

Motzord commented Aug 15, 2020

Боже храни лудука

@LudwigVonChesterfield LudwigVonChesterfield removed the Test Merge Candidate ПР с этим лэйблом будет или уже находится с тест мерже label Aug 19, 2020
@LudwigVonChesterfield
Copy link
Contributor

Нужно доделать.

@volas
Copy link
Member

volas commented Aug 26, 2020

UPD.
Блокировка техтоннелей возможна лишь в код не ниже красного. Снятие блокировки доступно в любой код.

ок, я снял свой дизлайк

@stale
Copy link

stale bot commented Sep 26, 2020

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

@stale stale bot added the Stalled PR label Sep 26, 2020
@stale
Copy link

stale bot commented Oct 3, 2020

ПР закрыт из-за длительного отсуствия активности. Для переоткрытия ПРа, пожалуйста, обратитесь к кому-либо из мейнтейнеров. Вы можете призвать их в комментарии слапнув @TauCetiStation/maintainers.

@stale stale bot closed this Oct 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Новая фича Stalled PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants