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

Добавление ручек для глав станции #412

Merged
merged 6 commits into from
Sep 6, 2024
Merged

Добавление ручек для глав станции #412

merged 6 commits into from
Sep 6, 2024

Conversation

Hyrmik
Copy link
Contributor

@Hyrmik Hyrmik commented Sep 6, 2024 •
edited
Loading

Описание PR

В игру для глав в их КПК были добавлены ручки по запросу mascotvalera (Discord)

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

Медиа

image

Проверки

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

Изменения

🆑 Хурмик

  • add: НТ расширила финансирование командования станции и в их КПК были добавлены особые ручки вместо обычных. Слава НТ!

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

github-actions bot commented Sep 6, 2024 •
edited
Loading

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

Resources/Textures/ADT/Objects/Misc/pens.rsi

State Old New Status
ADTpen_CE Added
ADTpen_QM Added
ADTpen_RD Added
ADTpen_cmo Added
ADTpen_hos Added

Edit: diff updated after 6cbae57

@Hyrmik Hyrmik requested review from 1Stepka1 and FaDeOkno September 6, 2024 03:35
Bolper
Bolper reviewed Sep 6, 2024
View reviewed changes
Resources/Prototypes/Entities/Objects/Devices/pda.yml
priority: -1
whitelist:
tags:
- Write
Copy link
Contributor

Choose a reason for hiding this comment

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

Ты поставил ADT tweak немного не так.
Он должен быть не на айдишнике предмета а в конце и начале изменений
По типу
ADT tweak starts
изменения
ADT tweak ends
Либо же ADT tweak ставится на единичном изменении в одну строчку

Copy link
Contributor

@Bolper Bolper Sep 6, 2024 •
edited
Loading

Choose a reason for hiding this comment

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

Типо в данном случае у тебя должно быть типо так
'
state: pda-qm
#adt tweak starts
penSlot:
startingItem: ADTPenQM
priority: -1
whitelist:
tags:
- Write
#adt tweak ends
- type: PdaBorderColor
'

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Окей

Bolper
Bolper approved these changes Sep 6, 2024
View reviewed changes
Resources/Prototypes/ADT/Entities/Objects/Misc/pen.yml
@@ -0,0 +1,67 @@
- type: entity
parent: Pen
Copy link
Contributor

Choose a reason for hiding this comment

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

Мог сразу преследоваться от PenEmbeddable, но как бы не критично, просто имей ввиду. (Как раз не пришлось бы добавлять лишние компоненты)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Проблема в том что я спрайты закинул в папку АДТ. А PenEmbeddable берёт спрайты не из АДТ. Поэтому и создал ещё одну, что бы брало правильное

Copy link
Contributor

Choose a reason for hiding this comment

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

Это можно было прописать~

Copy link
Contributor Author

Choose a reason for hiding this comment

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

_<

1Stepka1
1Stepka1 approved these changes Sep 6, 2024
View reviewed changes
Copy link
Collaborator

@1Stepka1 1Stepka1 left a comment

Choose a reason for hiding this comment

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

нойс

@Hyrmik Hyrmik added this pull request to the merge queue Sep 6, 2024
Merged via the queue into AdventureTimeSS14:master with commit b2036b8 Sep 6, 2024
23 checks passed
@Hyrmik Hyrmik deleted the pencil branch September 6, 2024 09:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Bolper Bolper Bolper approved these changes

@1Stepka1 1Stepka1 1Stepka1 approved these changes

@FaDeOkno FaDeOkno Awaiting requested review from FaDeOkno

Assignees
No one assigned
Labels
Changes: Localization Изменение локализации Changes: Sprites Изменение спрайтов No C# Status: Needs Review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Hyrmik @1Stepka1 @Bolper