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

[TM ONLY] Two SADAR kits #188

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

Conversation

MiraHell
Copy link

@MiraHell MiraHell commented Feb 28, 2025

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

Дает возможность взять кит садара два раза, не изменяя общее число специалистов

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

Локальные тесты

Changelog

🆑
add: Садары стронг
/:cl:

Обзор от Sourcery

Улучшения:

  • Позволяет брать наборы SADAR дважды.
Original summary in English

Summary by Sourcery

Enhancements:

  • Allows SADAR kits to be taken twice.

@ss220app ss220app bot added the 📜 CL валиден Этот чейнджлог будет успешно опубликован. label Feb 28, 2025
Copy link
Collaborator

@PhantornRU PhantornRU left a comment

Choose a reason for hiding this comment

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

Я вот не помню, нет ли там внутренней проверки на то что уже взята роль. Но ладно.
Запрос ВУЧа, не хотелось эти цифорки трогать

@MiraHell
Copy link
Author

Я вот не помню, нет ли там внутренней проверки на то что уже взята роль. Но ладно. Запрос ВУЧа, не хотелось эти цифорки трогать

Есть, она как раз available_vendor_num проверяет

@PhantornRU
Copy link
Collaborator

Тогда гуд)

@ularesen
Copy link

ularesen commented Mar 1, 2025

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

@PhantornRU
Copy link
Collaborator

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

не стоит, это уже перебор. Лучше уж добавить отдельную роль спека.

@MiraHell MiraHell changed the title [TM] Two SADAR kits [TM ONLY] Two SADAR kits Mar 1, 2025
@MeowEmiya

This comment has been minimized.

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