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

Exosuit DNA lock device #13190

Merged
merged 5 commits into from
Jul 3, 2024

Conversation

AirBlack
Copy link
Contributor

@AirBlack AirBlack commented Jun 22, 2024

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

Убрал по дефолту из всех рабочих мехов DNA lock, теперь чтобы лочить через ДНК надо печатать в робототехнике ДНК сканер для мехов. В мехах ОБР и синдиката по дефолту девайс присутствует
Также добавил требования доступа для дефолтного меха переработчиков и выключил ему по дефолту maint protocols и id upload panel.

Сделал залезание в мех через ду афтер. Добавил вытаскивание девайсов(ДНК сканер и трекер позиции) через откручивание их после откручивания батарейки.

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

Меня заебали переработчики блокирующие рипли на ДНК лок. Калеб козел.

Авторство

AirBlack - код идея и большие сочные яйца

Чеинжлог

🆑 AirBlack

  • tweak: Убран ДНК лок из мехов по дефолту в отдельный печатаемый ДНК сканер

@TauKitty
Copy link
Contributor

Changelog status: ✔️

@Deahaka
Copy link
Contributor

Deahaka commented Jun 22, 2024

Поставил лайк, а потом перечитал и там остаётся днк лок на нюкерских мехов. Дизлайк, отписка

@AirBlack
Copy link
Contributor Author

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

@Fluffy-510
Copy link
Contributor

Мне кажется, добавление требования доступа к меху переработчиков лишне

@AirBlack
Copy link
Contributor Author

Мне кажется, добавление требования доступа к меху переработчиков лишне

иди нафиг калеб какойто там который спиздил мех вчера, сдох взрывным имплантом и залочил рипли, когда робототехников не было и переработка встала

code/game/mecha/mecha.dm Outdated Show resolved Hide resolved
maps/centcom/centcom.dmm Outdated Show resolved Hide resolved
code/game/mecha/mecha.dm Outdated Show resolved Hide resolved
return

dna_lockable = TRUE
qdel(W)
Copy link
Member

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 Author

Choose a reason for hiding this comment

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

не согласен хранить внутри, напишу отдельное хранение если только вешать логику на сам итем, а для этого надо еще тогда аналоги написать с другим поведением, типа допустим тот который может несколько хранить например и всякая такая фигня, но у меня идей нет так что пусть будет итемом

code/game/mecha/mecha.dm Outdated Show resolved Hide resolved
code/game/mecha/mecha.dm Outdated Show resolved Hide resolved
code/game/mecha/mecha.dm Outdated Show resolved Hide resolved
Copy link
Member

@KIBORG04 KIBORG04 left a comment

Choose a reason for hiding this comment

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

А так, типа по мелочи изменить. но меня только немного напрягает, что ДНК Лок при установке в меху куделится, а при вытаскивании заново инстанциируется, но вроде пофиг

@AirBlack AirBlack requested a review from KIBORG04 July 3, 2024 14:46
Comment on lines +687 to +689
for(tracking)
actions += "Exosuit Tracker"
break
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
for(tracking)
actions += "Exosuit Tracker"
break
if(tracking)
actions += "Exosuit Tracker"

Эээ, не понял прикола

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Бля тупанул поторопился

@KIBORG04 KIBORG04 merged commit 512ef36 into TauCetiStation:master Jul 3, 2024
12 checks passed
TauKitty added a commit to TauCetiStation/ClassicChangelog that referenced this pull request Jul 3, 2024
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.

None yet

6 participants