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

AddMechToClothing #2265

Merged
merged 14 commits into from
Nov 23, 2024
Merged

AddMechToClothing #2265

merged 14 commits into from
Nov 23, 2024

Conversation

21Melkuu
Copy link

Описание PR

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

Техническая часть

Позволяет корректно добавлять MechComponent на одежду, добавляя механ рипли, если броня(экзоскелет) надевает игрок.
Ради ПР пришлось поправить некоторые моменты оригинального MechComponent
Так как специфика работы MechComponent добавляет все нужные кнопки, когда игрок становится пилотом рипли, пришлось скопипастить некоторую логику. Повторяющаяся логика - MechClothingComponent/system
Со слов адуча пофиксил матрешку рипли в рипли

Проверки

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

Изменения

🆑 21Gnomkuu", pulnet, mixnikita

  • add: Добавлен экзоскелет в шкафчик КМа.

@github-actions github-actions bot added Changes: Sprites Изменение затронуло файлы ".rsu / .png" Changes: Localization Изменение затронуло файлы ".ftl" Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" Changes: C# Изменение затронуло файлы ".cs" Merge Conflict Some content has conflicts. labels Nov 20, 2024
Copy link

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

Copy link

github-actions bot commented Nov 20, 2024

RSI Diff Bot; head commit fc769e9 merging into 000f9b0
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/SS220/Clothing/OuterClothing/Armor/mech_loader.rsi

State Old New Status
equipped-OUTERCLOTHING Added
icon Added
inhand-left Added
inhand-right Added

Resources/Textures/SS220/Interface/Actions/actions_mecha.rsi

State Old New Status
mech_grab Added

Edit: diff updated after fc769e9

@github-actions github-actions bot removed the Merge Conflict Some content has conflicts. label Nov 20, 2024
Copy link

@Ady4ik Ady4ik left a comment

Choose a reason for hiding this comment

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

обсуждали еще давно, так что одобрено.
Ждем одобрения по тех. части

Copy link
Collaborator

Choose a reason for hiding this comment

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

А вот сюда можно и копирайт

Copy link
Collaborator

Choose a reason for hiding this comment

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

И где?

@21Melkuu 21Melkuu requested a review from stalengd November 21, 2024 21:39
@canvaswalker
Copy link

мегакрут, хорош чувак!

@21Melkuu
Copy link
Author

@stalengd ну теперь тоооочно все

@stalengd stalengd merged commit 582b5b3 into SerbiaStrong-220:master Nov 23, 2024
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: C# Изменение затронуло файлы ".cs" Changes: Localization Изменение затронуло файлы ".ftl" Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" Changes: Sprites Изменение затронуло файлы ".rsu / .png"
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants