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

[MIRROR] Fix "All Within Theoretical Limits" not being granted when delamination variants change #1761

Merged
merged 1 commit into from
Jan 31, 2024

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Nova: NovaSector/NovaSector#693
Original PR: tgstation/tgstation#81168

About The Pull Request

I made a change in #80324 that prevents delamination variants from updating after the countdown is reached, however AWTL wouldn't obey this because the countdown proc sleeps while the crystal comes back from the countdown stage, so the delamination_strategy var will have changed. I removed 3 dead lines of code (because delam variants never change during the countdown).

Fixes #79528 this time hopefully

Why It's Good For The Game

work, damn it

Changelog

🆑 distributivgesetz
fix: All Within Theoretical Limits should properly unlock now when the crystal comes back from the countdown.
/:cl:

…ed when delamination variants change (#693)

* [NO GBP] Fix "All Within Theoretical Limits" not being granted when delamination variants change (#81168)

## About The Pull Request

I made a change in #80324 that prevents delamination variants from
updating after the countdown is reached, however AWTL wouldn't obey this
because the countdown proc sleeps while the crystal comes back from the
countdown stage, so the `delamination_strategy` var will have changed. I
removed 3 dead lines of code (because delam variants never change during
the countdown).

Fixes #79528 this time hopefully
## Why It's Good For The Game

work, damn it
## Changelog
:cl:
fix: All Within Theoretical Limits should properly unlock now when the
crystal comes back from the countdown.
/:cl:

* [NO GBP] Fix "All Within Theoretical Limits" not being granted when delamination variants change

---------

Co-authored-by: distributivgesetz <[email protected]>
@ReezeBL ReezeBL merged commit b764aa5 into master Jan 31, 2024
23 checks passed
@ReezeBL ReezeBL deleted the upstream-mirror-693 branch January 31, 2024 07:16
AnywayFarus added a commit that referenced this pull request Jan 31, 2024
Iajret pushed a commit that referenced this pull request Apr 2, 2024
* [NO GBP] Fixes prosthetic limb quirk preferences (#82354)

## About The Pull Request

Accidentally broke it during my latest quirk PR, oops!
## Changelog
:cl:
fix: Fixed Prosthetic limb quirk preferences
/:cl:

* [NO GBP] Fixes prosthetic limb quirk preferences

---------

Co-authored-by: DATAxPUNGED <[email protected]>
ReezeBL pushed a commit that referenced this pull request Apr 3, 2024
)

* [NO GBP] Fixes prosthetic limb quirk preferences (#82354)

## About The Pull Request

Accidentally broke it during my latest quirk PR, oops!
## Changelog
:cl:
fix: Fixed Prosthetic limb quirk preferences
/:cl:

* [NO GBP] Fixes prosthetic limb quirk preferences

---------

Co-authored-by: NovaBot <[email protected]>
Co-authored-by: DATAxPUNGED <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants