Skip to content

Delta Station Map Edit. Brig remap#12958

Merged
volas merged 40 commits intoTauCetiStation:masterfrom
Deahaka:delta_2_brig_popit-ka
Apr 29, 2024
Merged

Delta Station Map Edit. Brig remap#12958
volas merged 40 commits intoTauCetiStation:masterfrom
Deahaka:delta_2_brig_popit-ka

Conversation

@Deahaka
Copy link
Contributor

@Deahaka Deahaka commented Mar 15, 2024

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

Текстовое описание изменений в теме на форуме: https://forum.taucetistation.org/t/obsuzhdenie-remap-briga-no-na-delte/42456

StrongDMM-2024-03-15 06 58 24
StrongDMM-2024-03-29 20 10 44

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

Выполнение одного из пунктов TODO для Дельты в закрепах мап-канала дискорда

Авторство

some ideas from https://github.com/BeeStation/BeeStation-Hornet

Чеинжлог

🆑 Deahaka

  • map[link]: Восточная часть станции Дельта была изменена.

@TauKitty
Copy link
Contributor

Changelog status: ✔️

undock_act(/area/station/maintenance/chapel || /area/station/maintenance/bridge, "pod1")
undock_act(/area/station/maintenance/medbay || /area/station/maintenance/bridge || /area/station/civilian/gym, "pod2")
undock_act(/area/station/maintenance/dormitory || /area/station/maintenance/brig || /area/station/security/prison, "pod3")
undock_act(/area/station/maintenance/medbay || /area/station/maintenance/bridge || /area/station/maintenance/dormitory, "pod2")
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.

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

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
Member

Choose a reason for hiding this comment

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

вообще смотря на это, тут точно не всегда первый параметр будет? || возвращает первое TRUE, и тут /area/station/maintenance/medbay выглядит как path/путь и он сам по себе TRUE. Или это как-то по другому с зонами работает, и в бьенде путь к зоне == сама зона и я просто этого не знал? @simb11

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Да я это и добавил, теперь удаляю! Я сам не знаю как это работало, но все тесты эвакуации работали как надо.

@github-actions
Copy link

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

@Deahaka
Copy link
Contributor Author

Deahaka commented Apr 26, 2024

we are so back...

@volas volas merged commit 81c8024 into TauCetiStation:master Apr 29, 2024
TauKitty added a commit to TauCetiStation/ClassicChangelog that referenced this pull request Apr 29, 2024
@Deahaka Deahaka deleted the delta_2_brig_popit-ka branch April 29, 2024 18:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants