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.

Sign up for GitHub

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

Jump to bottom

ADD: Handheld radio now attached to Outer Clothing #2305

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

ADD: Handheld radio now attached to Outer Clothing #2305

wants to merge 1 commit into from

Conversation

ReeZer2
Copy link

@ReeZer2 ReeZer2 commented Dec 6, 2024 •
edited
Loading

Описание PR

Для всей верхней одежды была добавлена возможность крепить рацию.

Медиа

В этот слот:

image

Проверки

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

Изменения

🆑 ReeZii

  • add: Добавлена возможность крепить рацию в слот верхней одежды.

@github-actions github-actions bot added Changes: Localization Изменение затронуло файлы ".ftl" Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" labels Dec 6, 2024
@stalengd stalengd requested a review from Ady4ik December 6, 2024 16:12
@stalengd stalengd added the Need to be discussed Требуется обсуждение перед аппрувом/мёрджем label Dec 6, 2024
@UrPrice UrPrice removed the request for review from Ady4ik December 7, 2024 22:57
Kirus59
Kirus59 requested changes Dec 17, 2024
View reviewed changes
Resources/Prototypes/Entities/Clothing/OuterClothing/base_clothingouter.yml
Comment on lines +13 to +19
- type: ItemSlots
slots:
item:
name: clothing-outer-radio-handheld
whitelist:
tags:
- Radio
Copy link
Collaborator

@Kirus59 Kirus59 Dec 17, 2024 •
edited
Loading

Choose a reason for hiding this comment

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

А сам контейнер добавить?
Ну и в наследуемых энтити, где меняется ContainerContainer, также нужно будет добавить этот контейнер.

@Kirus59 Kirus59 self-assigned this Dec 17, 2024
@Kirus59 Kirus59 removed Need to be discussed Требуется обсуждение перед аппрувом/мёрджем Status: Needs Review labels Dec 17, 2024
@github-actions github-actions bot added the Merge Conflict Some content has conflicts. label Jan 15, 2025
@github-actions GitHub Actions
Copy link

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Kirus59 Kirus59 Kirus59 requested changes

@UrPrice UrPrice UrPrice approved these changes

Requested changes must be addressed to merge this pull request.

Assignees

@Kirus59 Kirus59

Labels
Changes: Localization Изменение затронуло файлы ".ftl" Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" Merge Conflict Some content has conflicts.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@ReeZer2 @UrPrice @Kirus59 @stalengd @Ady4ik