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

Возвращение Менторских ролей #50

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Zekins3366
Copy link

Описание PR

Были возвращены обратно Менторские роли для отделов, полный перечень:

  • Ведущий инженер
  • Ведущий врач
  • Ведущий учёный
  • Инструктор СБ

P.S. Данный ПР лишь добавляет роли в сборку, он не добавляет их на карты, а также не добавляет туда спавнеры ролей.

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

Некоторые люди хотели бы возвращения данных ролей, так почему нет?

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

https://discord.com/channels/919301044784226385/1306096119121580062

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

  • Добавлены роли
  • Добавлен перевод
  • Добавлены спавнеры
  • Изменены некоторые части кода для адекватно работы ролей
  • Убран(Закоммичен) контент связанный с вещами ведущий ролей у другой ролей, который был до этого вместо них соответственно

Медиа

Content Client_XsEIu1p2yn

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

  • add: Добавлены менторские роли

@Zekins3366 Zekins3366 requested a review from Morb0 as a code owner November 13, 2024 22:10
@github-actions github-actions bot added Status: Needs Review Changes: Localization Изменяет локализацию No C# labels Nov 13, 2024
@foboscheshir
Copy link

Та если не закрыть обратно рольки для новичков, менторы станут ролькой для чсв уёбков которые будут флексить тем что у них она открыта

Copy link
Collaborator

Choose a reason for hiding this comment

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

Почему бы просто не добавить их в спавнеры обычных ролей? Это избавит от проблем с картами.

@lexaSvarshik
Copy link

проклятие 220

@Tornado-Technology
Copy link
Collaborator

Я только думаю, что мемторские вещи из лодаута ролей ообычных убирать не стоит. Все же мемтор отличается от ученого, лишь названием профы.

Copy link

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot added the S: Merge Conflict У ПРа есть мердж конфликт label Dec 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Localization Изменяет локализацию No C# S: Merge Conflict У ПРа есть мердж конфликт Status: Needs Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants