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

[Feature] Половые запреты ♀️ ♂️ #985

Merged
merged 10 commits into from
Jan 15, 2025

Conversation

Schrodinger71
Copy link
Collaborator

@Schrodinger71 Schrodinger71 commented Jan 15, 2025

Описание PR

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

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

Добавление этой функциональности расширяет возможности настройки ролей, что может быть полезно в определённых ситуациях, например, для создания уникальных или специфичных для пола ролей.

Ссылка на публикацию в Discord:

Техническая информация

Добавлен класс SexRequirement, который является наследником JobRequirement. Этот класс позволяет задать список допустимых полов персонажа, и при попытке выбора роли будет проверяться, соответствует ли пол персонажа требованиям.
Код проверяет поле Sex персонажа и, в зависимости от настроек, либо разрешает, либо запрещает выбор роли.

Пример конфига в прототипе

    - !type:SexRequirement
      allowedSex:
      - Male
      - Unsexed
      - Female

Медиа

  1. Пример вайтлиста для женских персонажей:
    image

  2. Пример вайтлиста для мужских и бесполых персонажей:
    image

Требования

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

Критические изменения

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

Чейнджлог:
🆑 Шрёдька

  • add: Добавлена система ограничения по полу персонажа для ролей.

@Schrodinger71 Schrodinger71 marked this pull request as ready for review January 15, 2025 15:21
@github-actions github-actions bot added the Changes: Localization Изменение локализации label Jan 15, 2025
@Schrodinger71 Schrodinger71 changed the title Test [Feature] Половые запреты ♀️ ♂️ Jan 15, 2025
@Darkiich
Copy link
Contributor

Darkiich commented Jan 15, 2025

Описание PR

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

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

Добавление этой функциональности расширяет возможности настройки ролей, что может быть полезно в определённых ситуациях, например, для создания уникальных или специфичных для пола ролей.

Ссылка на публикацию в Discord:

Техническая информация

Добавлен класс SexRequirement, который является наследником JobRequirement. Этот класс позволяет задать список допустимых полов персонажа, и при попытке выбора роли будет проверяться, соответствует ли пол персонажа требованиям. Код проверяет поле Sex персонажа и, в зависимости от настроек, либо разрешает, либо запрещает выбор роли.

Пример конфига в прототипе

    - !type:SexRequirement
      allowedSex:
      - Male
      - Unsexed
      - Female

Медиа

  1. Пример вайтлиста для женских персонажей:
    image
  2. Пример вайтлиста для мужских и бесполых персонажей:
    image

Требования

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

Критические изменения

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

Чейнджлог: 🆑 Шрёдька

  • add: Добавлена проверка пола персонажа для роли с использованием нового класса SexRequirement, который позволяет задавать требования к полу для ролей.

Ты думаешь, что игрокам надо знать какой класс это делает? Прост думаю можно написать "Добавлена система ограничения по полу персонажа на роли"

@Schrodinger71
Copy link
Collaborator Author

Да окей, так лучше

Copy link
Contributor

@Darkiich Darkiich left a comment

Choose a reason for hiding this comment

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

Млядец-млядец, возьми с полки огурец)

@Schrodinger71 Schrodinger71 merged commit ccd83e9 into master Jan 15, 2025
14 of 15 checks passed
@Schrodinger71 Schrodinger71 deleted the feature-gender-filter-job branch January 15, 2025 19:06
@CrimeMoot
Copy link
Contributor

Нееет :c
Нормально он написал по факту и полезно! Хорошо!!1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Localization Изменение локализации S: Untriaged size/M
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants