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

FEAT: Роли новичков добавлены в консоль ГП #678

Merged
merged 5 commits into from
Nov 16, 2023

Conversation

PhantornRU
Copy link
Collaborator

Что этот PR делает

Консоль ГП теперь может навешивать работающую иконку новичковой роли и перекрашивать карту.
Модульность позволяет потенциально работать с этим и в будущем.

Почему это хорошо для игры

Да. Хорошо. Нужно больше набранных паровозиков кадетиков, а не то что мы сейчас видели, где ГП извращались с названиями. А ведь офицеры видят кадета-не-кадета, высказывают к ним требования как к офицерам и бедные кадетики перегружают свои головы и умирают задохнувшись от поступившей информации! Убережем кадетиков!

Изображения изменений

image

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

Да кому нужны эти тестирования когда есть такие прекрасные скриншоты. КОгда мы уже объединим эти 2 пункта.

Навешиваем роль, проверяем иконку:
image
image
image
image
image

Changelog

🆑
tweak: В консоли ГП теперь можно выдавать новичковые роли + перерисовывать их карты.
/:cl:

@ss220bot ss220bot added the 📜 CL валиден Этот чейнджлог будет успешно опубликован label Nov 12, 2023
@PhantornRU PhantornRU requested a review from Gaxeer November 13, 2023 12:59
@PhantornRU
Copy link
Collaborator Author

@Gaxeer повторно пересмотрел. Проблем не обнаружил

modular_ss220/jobs/code/cards_ids.dm Outdated Show resolved Hide resolved
modular_ss220/jobs/code/cards_ids.dm Outdated Show resolved Hide resolved
@PhantornRU PhantornRU requested a review from Gaxeer November 15, 2023 07:32
@PhantornRU PhantornRU merged commit 9e0db16 into ss220club:master Nov 16, 2023
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📜 CL валиден Этот чейнджлог будет успешно опубликован
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants