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.

Sign up for GitHub

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

Jump to bottom

[Port] Ghost Bar #136

Closed
wants to merge 12 commits into from
Closed

[Port] Ghost Bar #136

wants to merge 12 commits into from

Conversation

AwareFoxy
Copy link
Collaborator

@AwareFoxy AwareFoxy commented Dec 7, 2024 •
edited
Loading

Описание PR

Портирован призрачный бар с Goob Station
Goob-Station/Goob-Station#454

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

Для призраков

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

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

Медиа

2024-12-07-19-30-38.mp4

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

Список изменений

@github-actions github-actions bot added S: Untriaged S: Needs Review ПРу необходим ревью size/XL Changes: Map Изменяет карты Changes: Localization Изменяет локализацию Changes: UI and removed size/XL labels Dec 7, 2024
Vonsant
Vonsant requested changes Dec 7, 2024
View reviewed changes
Content.Client/UserInterface/Systems/Ghost/Widgets/GhostGui.xaml.cs Outdated Show resolved Hide resolved
Content.Client/UserInterface/Systems/Ghost/Widgets/GhostGui.xaml.cs Outdated Show resolved Hide resolved
Resources/Locale/ru-RU/_CorvaxNext/ghostbar/guidebook/ghost-gui.ftl Outdated
ghost-target-window-ghostbar = Призрачный Бар
ghost-window-spawn-ghostbar-button = Появится в баре
ghost-window-spawn-ghostbar-button-timer = Появится в баре через ({$time}с)
ghost-bar-rules = Относитесь к этой роли и станции так же, как к обычному Призраку. Вы можете говорить обо всех текущих событиях раунда без необходимости LOOC, и вы помните все из своей предыдущей жизни. НЕ нападайте на других, не затевайте драк и не пытайтесь сломать станцию. Если вы увидите, что кто-то делает это, пожалуйста, помогите, и он будет немедленно выброшен в космос. Также, если вы решите покинуть бар, вы НЕ БУДЕТЕ помнить ничего из того, что было здесь, или из вашей предыдущей жизни.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Бедная модерация при просмотре логов. И потом хер разберешь, обсуждал он IC в гостбаре или нет. Есть ли возможность добавлять к именам персонажей какой-нибудь префикс, чтобы потом по логам было точно видно, что перс в гостбаре?

Resources/Locale/ru-RU/_CorvaxNext/ghostbar/guidebook/ghost-gui.ftl Outdated Show resolved Hide resolved
Resources/Locale/ru-RU/_CorvaxNext/ghostbar/guidebook/ghost-gui.ftl Outdated Show resolved Hide resolved
Resources/Locale/ru-RU/_CorvaxNext/ghostbar/guidebook/ghost-gui.ftl Outdated Show resolved Hide resolved
Resources/Maps/_CorvaxNext/Nonstations/ghostbar.yml Show resolved Hide resolved
Resources/Prototypes/Goobstation/Entities/Markers/Spawners/ghost_roles.yml Show resolved Hide resolved
@CaptainSqrBeard
Copy link

Тестирование

  • Гостбар делает игроков невозрождаемыми. Если будешь это фиксить, я буду рад если эта система будет обобщённой.
  • Кто такой Icepick в бумажке в баре?
  • Я думаю лучше просто ограничить гриферскую активность в гостбаре. Тоесть отключить урон по игрокам, разрушаемость карты. На карте есть боксёрские перчатки, они должны продолжить работать. Тогда игрокам не придётся по приколу звать игроков в бар, если он OOC-шный.
  • Объекты на карте можно удалять, забирая их и гостаясь.
  • Объекты на карте можно бесконечно создавать, убирая их с персонажа и гостаясь

Карта

  • Атмос. Он реализован через механ (где-то там стоят трубы от генератора воздуха). Причём атмос не включен сразу же. В теории можно было-бы зафиксировать атмос, но я не знаю через какие компоненты это делается.
  • На карте есть несколько тайлов сетки решётки, что являются другими гридами.
  • Мусорка ведёт сразу-же в космос. Причём на том конце есть другая мусорка, и эта мусорка не имеет питания.
  • Двери ограничены доступом. Думаю их стоило сделать с общим доступом.
  • Карта и её грид(ы) не имеют названий, тяжело их распознать в админском списке.
  • На карте есть шлюзы в космос, что не имеют вентиляторов под собой. Можно легко спустить всю атмосферу.
    изображение

AwareFoxy
AwareFoxy commented Dec 7, 2024
View reviewed changes
Content.Client/UserInterface/Systems/Ghost/Widgets/GhostGui.xaml.cs Outdated Show resolved Hide resolved
Content.Client/UserInterface/Systems/Ghost/Widgets/GhostGui.xaml.cs Outdated Show resolved Hide resolved
Content.Client/UserInterface/Systems/Ghost/Widgets/GhostGui.xaml.cs Outdated Show resolved Hide resolved
Content.Client/UserInterface/Systems/Ghost/Widgets/GhostGui.xaml.cs Outdated Show resolved Hide resolved
Content.Client/UserInterface/Systems/Ghost/Widgets/GhostGui.xaml.cs Outdated Show resolved Hide resolved
Resources/Locale/ru-RU/_CorvaxNext/ghostbar/guidebook/ghost-gui.ftl Outdated Show resolved Hide resolved
Resources/Locale/ru-RU/_CorvaxNext/ghostbar/guidebook/ghost-gui.ftl Outdated Show resolved Hide resolved
Resources/Locale/ru-RU/_CorvaxNext/ghostbar/guidebook/ghost-gui.ftl Outdated Show resolved Hide resolved
FireNameFN
FireNameFN requested changes Dec 8, 2024
View reviewed changes
Copy link
Collaborator

@FireNameFN FireNameFN left a comment

Choose a reason for hiding this comment

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

ААААААААААААААААААААА

Resources/Locale/ru-RU/_CorvaxNext/ghostbar/guidebook/ghost-gui.ftl Show resolved Hide resolved
@Vonsant Vonsant added Undergoing Maintainer Discussion Этот PR в настоящее время проходит внутреннее обсуждение мейнтейнеров. S: DO NOT MERGE НЕ МЕРДЖИТЬ!!!! labels Dec 8, 2024
@Vonsant
Copy link
Collaborator

Vonsant commented Dec 9, 2024

Ожидаем компетентной карты от @Ko4ergaPunk

@Vonsant
Copy link
Collaborator

Vonsant commented Dec 9, 2024

ETA - 2 недели.

@AwareFoxy AwareFoxy marked this pull request as draft December 18, 2024 09:49
@AwareFoxy AwareFoxy closed this Dec 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Vonsant Vonsant Vonsant requested changes

@CaptainSqrBeard CaptainSqrBeard CaptainSqrBeard left review comments

@FireNameFN FireNameFN FireNameFN requested changes

@Tornado-Technology Tornado-Technology Awaiting requested review from Tornado-Technology Tornado-Technology is a code owner

Assignees
No one assigned
Labels
Changes: Localization Изменяет локализацию Changes: Map Изменяет карты Changes: UI S: DO NOT MERGE НЕ МЕРДЖИТЬ!!!! S: Needs Review ПРу необходим ревью S: Untriaged size/XL Undergoing Maintainer Discussion Этот PR в настоящее время проходит внутреннее обсуждение мейнтейнеров.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@AwareFoxy @CaptainSqrBeard @Vonsant @FireNameFN @Aidenkrz @Aviu00