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

Вещи ЦК и суперсинтезатор в лодауте музыканта. #712

Merged
merged 22 commits into from
Dec 6, 2024

Conversation

RipZoro1
Copy link
Contributor

@RipZoro1 RipZoro1 commented Oct 30, 2024

Описание PR

Вещи ЦК и суперсинтезатор в лодауте музыканта.

Почему / Баланс

Ссылка на публикацию в Discord

Техническая информация

Медиа

Требования

  • [Х] Я прочитал(а) и следую Руководство по созданию пулл реквестов. Я понимаю, что в противном случае мой ПР может быть закрыт по усмотрению мейнтейнера.
  • [Х] Я добавил скриншоты/видео к этому пулл реквесту, демонстрирующие его изменения в игре, или этот пулл реквест не требует демонстрации в игре

Критические изменения

Чейнджлог

🆑 Rip_Zoro

  • add: Добавлен пальто и сумка ПЦК
  • add: Вещи ПЦК добавлены в ЦентШкаф
  • add: Добавлен суперсинтезатор в лодаут музыканта за время

@github-actions github-actions bot added Changes: Sprites Изменение спрайтов No C# Changes: Localization Изменение локализации labels Oct 30, 2024
Copy link
Contributor

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

Copy link
Contributor

github-actions bot commented Oct 30, 2024

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

Resources/Textures/ADT/Clothing/Back/centcom_satchel.rsi

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

Resources/Textures/ADT/Clothing/OuterClothing/Coats/centcom_coat.rsi

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

Edit: diff updated after e965911

Copy link
Contributor

@Darkiich Darkiich left a comment

Choose a reason for hiding this comment

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

Так, во-первых спрайты должны быть в папке ADT

@github-actions github-actions bot added Status: Awaiting Changes Ожидание изменений and removed Status: Needs Review labels Oct 31, 2024
Copy link
Contributor

@Inconnu1337 Inconnu1337 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

@KashRas2 KashRas2 left a comment

Choose a reason for hiding this comment

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

Я боюсь потом забыть про этот пр, поэтому оставлю это просто как комментарий, а не требование.

Resources/Prototypes/Loadouts/Jobs/Civilian/musician.yml Outdated Show resolved Hide resolved
@jungarikjan
Copy link
Contributor

Суперсинтезатор не добавился нормально, Рип, я за тебя ща сделаю, окей?

@jungarikjan
Copy link
Contributor

image
Вот как у меня на локалке выглядит то же самое. Рип, извини, что влез, но ПР уже 4 дня висит, я хочу помочь

@jungarikjan jungarikjan self-assigned this Nov 10, 2024
@jungarikjan
Copy link
Contributor

Придется доделывать как будет время

@Darkiich Darkiich changed the title Вещи ЦК и суперсинтезатор в лодауте музыканта. Вещи ЦК Nov 20, 2024
@Darkiich Darkiich changed the title Вещи ЦК Вещи ЦК и суперсинтезатор в лодауте музыканта. Nov 20, 2024
@Darkiich Darkiich dismissed their stale review December 2, 2024 15:12

Собственно ревью доделал

@Darkiich Darkiich self-requested a review December 2, 2024 15:12
@github-actions github-actions bot added Status: Needs Review and removed Status: Awaiting Changes Ожидание изменений labels Dec 2, 2024
@Filokini
Copy link
Contributor

Filokini commented Dec 6, 2024

я помогал, так что ПР по любому ебейший, значит мержим.

@Filokini Filokini merged commit f2b6a55 into master Dec 6, 2024
15 checks passed
@Filokini Filokini deleted the schrodinger71-milashka branch December 6, 2024 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Localization Изменение локализации Changes: Sprites Изменение спрайтов No C#
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants