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

возвращаем к жизни гвард #13456

Merged
merged 3 commits into from
Aug 10, 2024

Conversation

NinjaPikachuska
Copy link
Contributor

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

из важного:
гвард не цеплялся к перезашедшему игроку и после переставал работать (#13212)

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

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

Авторство

Чеинжлог

@TauKitty
Copy link
Contributor

TauKitty commented Aug 9, 2024

Changelog status: 🔘 No changelog

Copy link
Contributor Author

@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.

вот они и плюсы навороченных идешников

code/modules/admin/topic.dm Outdated Show resolved Hide resolved
code/modules/admin/topic.dm Outdated Show resolved Hide resolved
@AzzyDreemurr13 AzzyDreemurr13 added the Admin Всё что касается админов. Логирование, инструментарий и т.д. label Aug 9, 2024
@volas
Copy link
Member

volas commented Aug 9, 2024

я ненавижу бьендовский client объект, в будущем хочу /datum/preferences/ как полноценный контейнер аналог клиенту оформить

@volas volas merged commit ee67520 into TauCetiStation:master Aug 10, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Admin Всё что касается админов. Логирование, инструментарий и т.д.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants