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

Expanded loadout for IAA #2099

Closed
wants to merge 6 commits into from
Closed

Expanded loadout for IAA #2099

wants to merge 6 commits into from

Conversation

Deenkaide
Copy link
Contributor

@Deenkaide Deenkaide commented May 3, 2024

Описание PR

Расширение снаряжения АВД шляпами, ботинками и перчатками. Все эти вещи были взяты из ЗаконШкафа.

(Мой первый ПР, поэтому прошу бить сильно, если что-то сделал не так)

Медиа

image
image
image

Проверки

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

Изменения

🆑 Deenkaide

  • add: Добавлен набор из трёх шляп в экипировку АВД
  • add: Добавлен набор из двух пар обуви в экипировку АВД
  • add: Добавлен набор из пяти пар перчаток в экипировку АВД

@github-actions github-actions bot added the Status: Needs Review This PR requires new reviews before it can be merged label May 3, 2024
@lzk228
Copy link
Collaborator

lzk228 commented May 3, 2024

в целом я бы для начала добавил перчи и тапки офам

@lzk228
Copy link
Collaborator

lzk228 commented May 3, 2024

РЯЯЯ НО У НИХ ЖЕ НЕТУ АВД

используйте для lawyera, а потом у нас просто впишем и будет заебумба

@github-actions github-actions bot added the Merge Conflict PR have unsolved merge conflicts label May 3, 2024
Copy link
Contributor

github-actions bot commented May 3, 2024

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

@Deenkaide
Copy link
Contributor Author

используйте для lawyera, а потом у нас просто впишем и будет заебумба

Ну... Я попробую к оффам переадресовать тогда, но объясните в чём конфликт, чтобы у оффов также не ошибиться

This reverts commit abc168d.
@github-actions github-actions bot removed the Merge Conflict PR have unsolved merge conflicts label May 3, 2024
@Deenkaide
Copy link
Contributor Author

Понял... я что-то воспринял за ошибку и фиксил то, чего нет)
Спасибо dezzzix за помощь

@Deenkaide
Copy link
Contributor Author

Закрываю PR, потому что отправил предложение руки и сердца оффам
space-wizards/space-station-14#27653

@Deenkaide Deenkaide closed this May 3, 2024
@Deenkaide Deenkaide reopened this May 6, 2024
@Deenkaide
Copy link
Contributor Author

Чтож... оффы отклонили мой ПР в полном объёме из-за того, что это им не подходит

@Deenkaide
Copy link
Contributor Author

Вроде всё окончательно готово, если есть какие-то недочёты - исправлю

@lzk228
Copy link
Collaborator

lzk228 commented May 8, 2024

ну, мнение получено, я пр не одобряю

If people wish to have these items, they should simply seek them out from various vendors at the start of the round. Loadouts should be reserved for things that are actually specific to jobs and not just random clothing. There's a reason why you can't just equip any arbitrary jumpsuit in a loadout.

@Deenkaide
Copy link
Contributor Author

Ну чтож, тогда пора удалять половину контента из существующих лодаутов.

Я попробую сегодня починить ПР и буду ждать Морти

@Morb0
Copy link
Member

Morb0 commented May 8, 2024

ну, мнение получено, я пр не одобряю

If people wish to have these items, they should simply seek them out from various vendors at the start of the round. Loadouts should be reserved for things that are actually specific to jobs and not just random clothing. There's a reason why you can't just equip any arbitrary jumpsuit in a loadout.

Я согласен тоже с этим мнением, для такого достаточно вендормата

@Deenkaide
Copy link
Contributor Author

ну, мнение получено, я пр не одобряю

If people wish to have these items, they should simply seek them out from various vendors at the start of the round. Loadouts should be reserved for things that are actually specific to jobs and not just random clothing. There's a reason why you can't just equip any arbitrary jumpsuit in a loadout.

Я согласен тоже с этим мнением, для такого достаточно вендормата

Это странно, что у учёных и т.д. есть и береты, и ассортимент перчаток, но АВД нельзя...

@Morb0
Copy link
Member

Morb0 commented May 8, 2024

ну, мнение получено, я пр не одобряю

If people wish to have these items, they should simply seek them out from various vendors at the start of the round. Loadouts should be reserved for things that are actually specific to jobs and not just random clothing. There's a reason why you can't just equip any arbitrary jumpsuit in a loadout.

Я согласен тоже с этим мнением, для такого достаточно вендормата

Это странно, что у учёных и т.д. есть и береты, и ассортимент перчаток, но АВД нельзя...

Хм... если прям такой набор перчаток...

@Deenkaide Deenkaide closed this by deleting the head repository Jun 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Needs Review This PR requires new reviews before it can be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants