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: Identity concealment for chat and radio messages #2618

Conversation

ReeZer2
Copy link

@ReeZer2 ReeZer2 commented Feb 23, 2025

Описание PR

По таску из борды.
https://github.com/orgs/SerbiaStrong-220/projects/27/views/1?pane=issue&itemId=98969068

  1. Персонаж без ID карты и без блокера:

    • Личность не скрыта.

    • Возвращается имя ентити.

  2. Персонаж с ID картой, но без имени:

    • Личность скрыта.

    • Возвращается описание (например, "Человек среднего возраста").

  3. Персонаж с блокером (маской):

    • Личность скрыта.

    • Возвращается описание (например, "Человек среднего возраста").

  4. Персонаж с ID картой и именем:

    • Личность не скрыта.

    • Возвращается имя с ID карты.

  5. Рация:

    • Если ID карта есть и имя не пустое, возвращается имя с ID карты.

    • Если ID карты нет или имя пустое, возвращается "Неизвестный".

Медиа

Безымянный-2024-11-14-0745

Проверки

  • PR полностью завершён и мне не нужна помощь чтобы его закончить.
  • Я ознакомился с наставлениями по работе с репозиторием и следовал им при создании PR'а.
  • Я внимательно просмотрел все свои изменения и багов в них не нашёл.
  • Я запускал локальный сервер со своими изменениями и всё протестировал.
  • Я добавил скриншот/видео демонстрации PR в игре, или этот PR этого не требует.

Изменения

🆑 ReeZii

  • add: Добавлено сокрытие личности в чате и рации при сокрытии лица или ID карты

@ReeZer2 ReeZer2 requested a review from UrPrice February 23, 2025 00:35
@github-actions github-actions bot added Changes: Localization Изменение затронуло файлы ".ftl" Changes: C# Изменение затронуло файлы ".cs" Status: Needs Review labels Feb 23, 2025
@ReeZer2 ReeZer2 requested a review from Ady4ik February 23, 2025 00:36
@ReeZer2
Copy link
Author

ReeZer2 commented Feb 23, 2025

@UrPrice прикрепить бы таск к борде, и вдруг еще чет надо, весь функционал в описании рассписан

@UrPrice
Copy link
Collaborator

UrPrice commented Feb 23, 2025

@UrPrice прикрепить бы таск к борде, и вдруг еще чет надо, весь функционал в описании рассписан

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

@UrPrice
Copy link
Collaborator

UrPrice commented Feb 23, 2025

{27F9F0AB-D02C-433B-8908-D6D3D8EDFD1B}

Кейс: Персонаж с блокером и чужой ID картой.
Как я понимаю, должно даваться имя ентити с ID карты из-за блокера, но вероятно не продумали это в ТЗ.

Сейчас это было бы аналогично, что с блокером без ID карты бы писалось имя чара, но мы же скрываем. А значит с блокером и чужой ID-шкой - должны подменять.

Собственно для рации это тоже диссонанс:
dotnet_zLTiAvDIf5

Но с другой стороны. Если блокера нет, то рация должна подхватывать изменённую личность, а просто сейлог - оригинальную.

@ReeZer2
Copy link
Author

ReeZer2 commented Feb 23, 2025

Чеки апрува :(

@UrPrice
Copy link
Collaborator

UrPrice commented Feb 23, 2025

Чеки апрува :(

Забей, я его сломал, завтра либо починю, либо отключу снова

@ReeZer2 ReeZer2 requested a review from stalengd February 23, 2025 09:58
@ReeZer2
Copy link
Author

ReeZer2 commented Feb 23, 2025

@stalengd Вроде всё что мог инвертнуть - инвертнул, сократить - сократил

@stalengd stalengd merged commit 062ee44 into SerbiaStrong-220:master Feb 23, 2025
13 of 14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: C# Изменение затронуло файлы ".cs" Changes: Localization Изменение затронуло файлы ".ftl"
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants