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

engineering-update-thruster #2338

Merged
merged 10 commits into from
Dec 15, 2024

Conversation

EstKemran
Copy link

@EstKemran EstKemran commented Dec 10, 2024

Описание PR

1 - Манёвровый двигатель - малый настенный двигатель, для совершения манёвров в космосе. Мощность где-то 1\3 от исходного;

2 - Большой двигатель (Разделён на 3 части) - комплекс усиленных двигателей, используется для оснащенья тяжелых космических объектов;
Левый и Правый мощнее стандартного на 0.5 от исходного;
Центральны мощнее в 2 раза от исходного;

Медиа
imag12e
truster_mini

Проверки

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

Изменения
🆑 MIXnikita, Kirus

  • add: Добавлены два новых двигателя: маневренный и большой.

@github-actions github-actions bot added Changes: Sprites Изменение затронуло файлы ".rsu / .png" Changes: Localization Изменение затронуло файлы ".ftl" Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" labels Dec 10, 2024
Copy link

github-actions bot commented Dec 10, 2024

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

Resources/Textures/SS220/Structures/Shuttles/thruster_big_center.rsi

State Old New Status
base Added
thrust Added
thrust_burn_unshaded Added

Resources/Textures/SS220/Structures/Shuttles/thruster_big_left.rsi

State Old New Status
base Added
thrust Added
thrust_burn_unshaded Added

Resources/Textures/SS220/Structures/Shuttles/thruster_big_right.rsi

State Old New Status
base Added
thrust Added
thrust_burn_unshaded Added

Resources/Textures/SS220/Structures/Shuttles/thruster_shunting.rsi

State Old New Status
base Added
frame-base Added
thrust Added
thrust_burn_unshaded Added

Edit: diff updated after de8e3f6

@EstKemran
Copy link
Author

EstKemran commented Dec 10, 2024

Есть баг, при постройке манёвренный двигатель разворачивается на 180 градусов.

Можно либо как есть замёрджить для мапперов, если им срочно нужно,
либо замёрджить вместе с другими engineering-update`s, как их закончку.

@EstKemran EstKemran changed the title Engineering update thruster engineering-update-thruster Dec 10, 2024
@UrPrice
Copy link
Collaborator

UrPrice commented Dec 11, 2024

Есть баг, при постройке манёвренный двигатель разворачивается на 180 градусов.

Можно либо как есть замёрджить для мапперов, если им срочно нужно, либо замёрджить вместе с другими engineering-update`s, как их закончку.

Пофиксил? Там можно ротейт в принципе в рецепте постройки вроде изменять либо делать его таким, какой игроком запланирован.

@github-actions github-actions bot added the Merge Conflict Some content has conflicts. label Dec 12, 2024
Copy link

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

@github-actions github-actions bot removed the Merge Conflict Some content has conflicts. label Dec 12, 2024
@EstKemran
Copy link
Author

Есть баг, при постройке манёвренный двигатель разворачивается на 180 градусов.
Можно либо как есть замёрджить для мапперов, если им срочно нужно, либо замёрджить вместе с другими engineering-update`s, как их закончку.

Пофиксил? Там можно ротейт в принципе в рецепте постройки вроде изменять либо делать его таким, какой игроком запланирован.

Плюс минус готово

Copy link
Collaborator

@Kirus59 Kirus59 left a comment

Choose a reason for hiding this comment

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

Направление маневрового двигателя правильно считывает (ну, в какую сторону толкать)?

@EstKemran
Copy link
Author

Направление маневрового двигателя правильно считывает (ну, в какую сторону толкать)?

Ага

@UrPrice UrPrice removed the request for review from Ady4ik December 15, 2024 00:23
@UrPrice
Copy link
Collaborator

UrPrice commented Dec 15, 2024

bambam.mp4

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

Successfully merging this pull request may close these issues.

4 participants