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

[Port] Advanced Prying #82

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

[Port] Advanced Prying #82

wants to merge 3 commits into from

Conversation

PuroSlavKing
Copy link

Описание PR

Taken from: WWhiteDreamProject/wwdpublic#31
Мгновенное открытие обесточенных шлюзов (Повышение динамики + SS13 Reference) инструментами и повышение полезности топора.

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

Особо не повлияет.

Ссылка на ветку

Технические детали

Медиа

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

Список изменений

🆑 Warete

  • tweak: Изменено открытие обесточенных шлюзов, теперь при помощи инструментов они открываются и закрываются мгновенно.
  • tweak: Добавлена возможность срывать тайлы покрытия, при помощи топора.

PuroSlavKing and others added 3 commits November 25, 2024 14:33
@PuroSlavKing @RedBurningPhoenix
@NairSark @Spatison
[Port] Advanced Prying (space-syndicate#18) (space-syndicate#760)
d31fa54 
* [Port] Advanced Prying (space-syndicate#31)

* balance: Less rcds, more axe functions (space-syndicate#132)

* Revert "Adds RCD and RCD ammo to all engineer lockers. (#22688)"

This reverts commit 4e94a27.

* add: Axe can pry platings. Again.

* fix: RCD in CE locker

* Make Axe great again

---------



* [Tweak] Prying / Вскрытие (space-syndicate#43)

* tweak: neglecting time when using scrap

* fix

* fix

* fix

* fixik

---------

Co-authored-by: RedBurningPhoenix <147742474+RedBurningPhoenix@users.noreply.github.com>
Co-authored-by: Nair Sark <26607408+NairSark@users.noreply.github.com>
Co-authored-by: Spatison <137375981+Spatison@users.noreply.github.com>
@PuroSlavKing
remark
1a5701e
@PuroSlavKing
fix dublicate
378cbb8
@PuroSlavKing PuroSlavKing requested a review from Morb0 as a code owner November 25, 2024 12:13
@github-actions github-actions bot added S: Needs Review ПРу необходим ревью S: Untriaged size/M Changes: Localization Изменяет локализацию and removed size/M labels Nov 25, 2024
@CaptainSqrBeard
Copy link

Тестирование

  • Плитка снимается ломом моментально. Это офигенно, когда что-то строишь и тебе надо поменять пол. Хотя и внешний вид станции теперь тоже легко гриферится. За пол минуты можно сделать то, что я сделал на скриншоте, а то и быстрее.
  • Покрытие (тайлы без плитки) тоже снимается мгновенно, но только при помощи топора. Вот это уже не очень хорошо. Гриферить атмосферу на станции ещё никогда не было так просто.
  • Челюсти жизни не снимают плитку моментально, вероятно это баг.

изображение

FireNameFN
FireNameFN requested changes Nov 26, 2024
View reviewed changes
Copy link
Collaborator

@FireNameFN FireNameFN left a comment

Choose a reason for hiding this comment

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

И так везде

Content.IntegrationTests/Tests/DoAfter/DoAfterCancellationTests.cs
@@ -66,6 +66,15 @@ public async Task CancelTilePry()
{
await SetTile(Floor);
await InteractUsing(Pry, awaitDoAfters: false);

// CORVAX-NEXT START
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
// CORVAX-NEXT START
// Corvax-Next-AdvancedPrying-Start

@CrimeMoot
Copy link

CrimeMoot commented Nov 26, 2024 •
edited
Loading

Челюсти жизни моментально открываются запитанные/заблокированные шлюзы. Синдикатовские.
Хорошая замена емагу. Ещё и стоит 2тк

@CaptainSqrBeard
Copy link

Челюсти жизни моментально открываются запитанные/заблокированные шлюзы. Синдикатовские.
Хорошая замена емагу. Ещё и стоит 2тк

Нету такого. Запитанные двери всё ещё открываются с длинной задержкой.

@CrimeMoot
Copy link

CrimeMoot commented Nov 27, 2024 •
edited
Loading

Нету такого. Запитанные двери всё ещё открываются с длинной задержкой.

Все да. Но почему-то синдикатовские так не думают.

@PuroSlavKing
Copy link
Author

Нету такого. Запитанные двери всё ещё открываются с длинной задержкой.

Все да. Но почему-то синдикатовские так не думают.

Потому-что у них скорость вскрытия на 1.5 больше чем у обычных, буквально недавно подняли эту тему на бекмене, тебе ли не знать

@github-actions github-actions bot added the S: Merge Conflict У ПРа есть мердж конфликт label Dec 5, 2024
@github-actions GitHub Actions
Copy link

github-actions bot commented Dec 5, 2024

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

@FireNameFN FireNameFN FireNameFN requested changes

@Morb0 Morb0 Awaiting requested review from Morb0

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
Changes: Localization Изменяет локализацию S: Merge Conflict У ПРа есть мердж конфликт S: Needs Review ПРу необходим ревью S: Untriaged
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@PuroSlavKing @CaptainSqrBeard @CrimeMoot @FireNameFN