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

add: True-Observing port #6665

Conversation

timchigaming
Copy link
Contributor

@timchigaming timchigaming commented Mar 15, 2025

Описание

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

Причина создания ПР / Почему это хорошо для игры

6 голосов ЗА на уже истёкшей(ну и ладно) предложке: https://discord.com/channels/617003227182792704/755125334097133628/1261084045878624338
В целом фича очень полезная как для администрирующих сервер, так и для игроков чтоб можно точно знать "Надо ли на этого гада жалобу за самоантагонизм писать, или нет?"

Демонстрация изменений

[Видосик большой для гитхаба]
https://cdn.discordapp.com/attachments/734823601110515882/1350399349749977138/dreamseeker_GBabHzLaDx.mp4?ex=67d698e6&is=67d54766&hm=76e93c0b5996f98d382e9f036e0ccf7696d39f741398608e741f8b9830103fa4&

image

Тесты

На локалке тесты были, хорошие тесты, плохим не пахло.
Если будут репорты - починим.

j33R2WLgGe.mp4

@github-actions github-actions bot added TGUI Affects TGUI javascript Pull requests that update Javascript code Merge Conflict This doesn't seem right labels Mar 15, 2025
@timchigaming timchigaming requested a review from Den109G March 15, 2025 20:06
@timchigaming timchigaming marked this pull request as ready for review March 15, 2025 20:07
@timchigaming timchigaming force-pushed the Попытка-номер-8---True-Observing-porting branch from 22af62e to 682fe61 Compare March 16, 2025 10:08
@github-actions github-actions bot removed the Merge Conflict This doesn't seem right label Mar 16, 2025
Всё работает, еще 0.5 процента на поправки от ревьюверов
@github-actions github-actions bot removed the Merge Conflict This doesn't seem right label Apr 1, 2025
@BeebBeebBoob BeebBeebBoob merged commit c8265fb into ss220-space:master220 Apr 1, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DMI Affects DMI files javascript Pull requests that update Javascript code TGUI Affects TGUI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants