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

a lot of fixes #375

Merged
merged 3 commits into from
Aug 30, 2024
Merged

a lot of fixes #375

merged 3 commits into from
Aug 30, 2024

Conversation

KashRas2
Copy link
Collaborator

@KashRas2 KashRas2 commented Aug 30, 2024 •
edited
Loading

Описание PR

По сути фикс всех моих апдейтов.

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

Проверки

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

Изменения

🆑 KashRas2

  • add: Был возвращен пульт от пожарных сигнализаций атмосферным техникам.
  • tweak: Реакция получения заукера теперь имеет температурный лимит не в 75к К, а в 100к К.
  • tweak: Респрайт банкоматов и цветных комбинезонов.
  • fix: Исправлена невозможность строительства воздушных сигнализаций на стенах при использовании РРТ.

@github-actions github-actions bot added Changes: UI Изменение интерфейса Changes: Sprites Изменение спрайтов Changes: Localization Изменение локализации labels Aug 30, 2024
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Aug 30, 2024 •
edited
Loading

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

Resources/Textures/Clothing/Uniforms/Jumpskirt/color.rsi

State Old New Status
equipped-INNERCLOTHING Modified
icon Modified

Resources/Textures/Clothing/Uniforms/Jumpsuit/color.rsi

State Old New Status
equipped-INNERCLOTHING Modified
icon Modified

Resources/Textures/ADT/Structures/Machines/atm.rsi

State Old New Status
off Removed
on Removed
atm Added
atm_off Added

Edit: diff updated after 785a2cd

@Schrodinger71
Copy link
Collaborator

Как дома буду мб гляну

Schrodinger71
Schrodinger71 approved these changes Aug 30, 2024
View reviewed changes
Copy link
Collaborator

@Schrodinger71 Schrodinger71 left a comment

Choose a reason for hiding this comment

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

НННууу наверное ок

@Schrodinger71
Copy link
Collaborator

Особо замечаний нет к КашРашу

@Schrodinger71 Schrodinger71 requested a review from Bolper August 30, 2024 19:00
@Bolper
Copy link
Contributor

Bolper commented Aug 30, 2024

Ааааа

Bolper
Bolper reviewed Aug 30, 2024
View reviewed changes
Content.Server/ADT/Atmos/Reactions/HyperNobliumProductionReaction.cs
@@ -10,10 +10,6 @@ public sealed partial class HyperNobliumProductionReaction : IGasReactionEffect
{
public ReactionResult React(GasMixture mixture, IGasMixtureHolder? holder, AtmosphereSystem atmosphereSystem, float heatScale)
{
var initialHyperNoblium = mixture.GetMoles(Gas.HyperNoblium);
Copy link
Contributor

Choose a reason for hiding this comment

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

Ура

Resources/Prototypes/ADT/Atmospherics/reactions.yml
@@ -202,7 +202,7 @@
id: ZaukerProduction
priority: 9
minimumTemperature: 50000
maximumTemperature: 75000
maximumTemperature: 100000
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
Collaborator

Choose a reason for hiding this comment

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

нет не ошибся он

@1Stepka1
Copy link
Collaborator

CL написано довольно странно и пойдет если только внутри команды,а для общего чейджлога как-то не очень,лучше будет немного изменить и сделать более точные формулировки изменений

@KashRas2
Copy link
Collaborator Author

CL написано довольно странно и пойдет если только внутри команды,а для общего чейджлога как-то не очень,лучше будет немного изменить и сделать более точные формулировки изменений

Поправил если что.

@KashRas2 KashRas2 added this pull request to the merge queue Aug 30, 2024
Merged via the queue into AdventureTimeSS14:master with commit 525e033 Aug 30, 2024
22 checks passed
@KashRas2 KashRas2 deleted the new-feature branch August 30, 2024 21:49
@KashRas2 KashRas2 restored the new-feature branch August 30, 2024 21:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Bolper Bolper Bolper left review comments

@Schrodinger71 Schrodinger71 Schrodinger71 approved these changes

@1Stepka1 1Stepka1 Awaiting requested review from 1Stepka1

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

Successfully merging this pull request may close these issues.
5 participants
@KashRas2 @Schrodinger71 @Bolper @1Stepka1 @KashRas