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] Removes virtual-specific megafauna, fixes boss map issues #415

Merged
merged 1 commit into from
Nov 3, 2023

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Skyrat: Skyrat-SS13/Skyrat-tg#24737
Original PR: tgstation/tgstation#79424

About The Pull Request

Another atomization of the content patch-
Let's not subtype everything into virtual variants

This also fixes some bugs that were present with vdom boss maps

  • Made the bubblegum map larger so he wouldn't teleport out
  • Legion is just far too annoying to have to fix code wise, given that few if anyone tries it with the base virtual equipment
what do you mean too annoying?

Legion doesn't handle dying like the standard megafauna - each time it "dies" it looks to see if there's another legion in the entire game world. Given the real one very likely exists, this basically means it won't drop anything. I'd have to modify legion's death proc specifically for the virtual domain. No other megafauna has this level of weirdness (nor splitting behavior) and coupled with the fact I think no one tries the domain, I'm just removing it.

Why It's Good For The Game

Cleans up megafauna subtypes that were specific for the virtual domain. Now, we won't need to keep adding to the list if there's ever a new one
Fixes #79203
Fixes #79200

Changelog

🆑 jlsnow301
fix: Bubblegum should no longer teleport out of the simulation when threatened
del: Chamber of Echoes map removed as it conflicts with the actual Legion
/:cl:

…O GBP] [MDB IGNORE] (#24737)

* Removes virtual-specific megafauna, fixes boss map issues [NO GBP] (#79424)

## About The Pull Request
Another atomization of the content patch-
Let's not subtype everything into virtual variants

This also fixes some bugs that were present with vdom boss maps
- Made the bubblegum map larger so he wouldn't teleport out
- Legion is just far too annoying to have to fix code wise, given that
few if anyone tries it with the base virtual equipment

<details>
<summary>what do you mean too annoying?</summary>

Legion doesn't handle dying like the standard megafauna - each time it
"dies" it looks to see if there's another legion in the entire game
world. Given the real one very likely exists, this basically means it
won't drop anything. I'd have to modify legion's death proc specifically
for the virtual domain. No other megafauna has this level of weirdness
(nor splitting behavior) and coupled with the fact I think no one tries
the domain, I'm just removing it.

</details>

## Why It's Good For The Game
Cleans up megafauna subtypes that were specific for the virtual domain.
Now, we won't need to keep adding to the list if there's ever a new one
Fixes #79203
Fixes #79200

## Changelog
:cl:
fix: Bubblegum should no longer teleport out of the simulation when
threatened
del: Chamber of Echoes map removed as it conflicts with the actual
Legion
/:cl:

* Removes virtual-specific megafauna, fixes boss map issues [NO GBP]

---------

Co-authored-by: Jeremiah <[email protected]>
@ReezeBL ReezeBL merged commit f74f611 into master Nov 3, 2023
24 checks passed
@ReezeBL ReezeBL deleted the upstream-mirror-24737 branch November 3, 2023 07:03
AnywayFarus added a commit that referenced this pull request Nov 3, 2023
Iajret pushed a commit that referenced this pull request Jan 11, 2024
* Fix supermatter component runtime on non-supermatters (#80894)

## About The Pull Request

Used the wrong var for this istype check.

* Fix supermatter component runtime on non-supermatters

---------

Co-authored-by: MrMelbert <[email protected]>
Co-authored-by: NovaBot <[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