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] Material pouch #1074

Merged
merged 10 commits into from
Mar 11, 2024
Merged

[FEAT] Material pouch #1074

merged 10 commits into from
Mar 11, 2024

Conversation

Chorden523
Copy link

@Chorden523 Chorden523 commented Mar 10, 2024

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

Краткое описание:
Добавляет material pouch в игру. Получить можно из биогена

Детальное описание:
Это подсумок для хранения листов материалов, вмещает до 5 полных стопок. Можно поместить только в карман либо на пояс. Может держать только сталь, стекло, плазму, титан и т.д. Спрайт взят со скайрата.

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

Оживляем строительство вручную и добавляем разнообразие для инженеров

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

output-onlinepngtools
Screenshot_458

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

Запустил сервер, проверил поднимание предметов - работает. Добавил в биоген, запустил - выдаётся без проблем.

Changelog

🆑
add: Добавлен material pouch в биогенератор. Это подсумок для переноски материалов, помещается только в карман и на пояс. Вмещает до 5 стопок материалов.
/:cl:

@github-actions github-actions bot added the 🖌️ Спрайты Вы заработали свою миска-рис и кошко-жена. Партия гордится вами! label Mar 10, 2024
@ss220bot ss220bot added the 📜 CL валиден Этот чейнджлог будет успешно опубликован label Mar 10, 2024
modular_ss220/objects/_objects.dm Outdated Show resolved Hide resolved
@dj-34
Copy link
Collaborator

dj-34 commented Mar 10, 2024

Зачем нам дубликат, если на оффах уже есть construction bag, который даже изображен на твоем скриншоте
image

@Chorden523
Copy link
Author

В ПРе написано, что material pouch может держать только материалы, а construction bag только платы может держать и батарейки с проводами. Так что это две разные вещи, хоть с виду и похожи

@m-dzianishchyts
Copy link
Collaborator

по 250 листов металла в каждом кармане, найс

@dj-34
Copy link
Collaborator

dj-34 commented Mar 10, 2024

В ПРе написано, что material pouch может держать только материалы, а construction bag только платы может держать и батарейки с проводами. Так что это две разные вещи, хоть с виду и похожи

чзх, ща перепроверю и вернусь

@m-dzianishchyts
Copy link
Collaborator

я предлагаю запретить вообще куда-то складывать, только в руках

Copy link
Collaborator

@dj-34 dj-34 left a comment

Choose a reason for hiding this comment

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

Ладно, construction bag не вмещает материалы.
Однако, я пожалуй выступлю против этого ПРа, так как у нас есть тележка для материалов, которую оффы кстати недавно обновили...
Уж больно сильный бафф для инженеров.

@dj-34 dj-34 added :trollface: Фича Новая функция, фишка или механика. Абсолютно точно новые баги. 🕛 Ждёт мержа Совсем скоро это будет залито labels Mar 10, 2024
@Legendaxe Legendaxe merged commit 61f971a into ss220club:master Mar 11, 2024
13 checks passed
@Chorden523 Chorden523 deleted the mat_pouch branch March 11, 2024 08:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🕛 Ждёт мержа Совсем скоро это будет залито 🖌️ Спрайты Вы заработали свою миска-рис и кошко-жена. Партия гордится вами! 📜 CL валиден Этот чейнджлог будет успешно опубликован :trollface: Фича Новая функция, фишка или механика. Абсолютно точно новые баги.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants