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

[PORT] Fixing two fishing issues in one PR. #5532

Conversation

LikeLakers2
Copy link
Collaborator

@LikeLakers2 LikeLakers2 commented Feb 18, 2025

About The Pull Request

This PR partially ports the following PR from tgstation:

This should fix #5520. Mobs who drop into a chasm in a virtual domain will no longer be retrievable. This also incidentally splits up the "chasm fallen mob" list into different z-level categories - meaning that mobs which fall in chasms at centcom can't be retrieved from a chasm on the station; simultaneously, mobs who fall into a station-side chasm can be retrieved from any station chasm.

The PR was not fully ported - part of it relied on changes made in tgstation/tgstation#80599, which is a balance change that has not been ported yet.

Why It's Good For The Game

Fixes #5520

Changelog

🆑MichiRecRoom, Ghommie
fix: (Ghommie) Fixed a few harddel issues with mob spawns that caused charred corpses fished from lavaland to create an invisible blockade.
fix: (Ghommie) Fixed being able to fish up mobs that have fallen in totally different z-levels with a rescue hook (i.e. from bitrunning domains to lavaland).
/:cl:

This will fix #83730, fix #85985 and fix #86033.

See above.

:cl:
fix: You can no longer pickup closets and crates wrapped in a package
with a fishing rod.
fix: Fixed a few harddel issues with mob spawns that caused charred
corpses fished from lavaland to create an invisible blockade.
fix: Fixed being able to fish up mobs that have fallen in totally
different z-levels with a rescue hook (i.e. from bitrunning domains to
lavaland).
/:cl:

---------

Co-authored-by: MrMelbert <51863163+MrMelbert@users.noreply.github.com>
@LikeLakers2 LikeLakers2 changed the title [PORT] Fixing three fishing issues in one PR. (#86042) [PORT] Fixing ~~three~~ two fishing issues in one PR. (#86042) Feb 18, 2025
@LikeLakers2 LikeLakers2 changed the title [PORT] Fixing ~~three~~ two fishing issues in one PR. (#86042) [PORT] Fixing two fishing issues in one PR. (#86042) Feb 18, 2025
@LikeLakers2 LikeLakers2 changed the title [PORT] Fixing two fishing issues in one PR. (#86042) [PORT] Fixing two fishing issues in one PR. Feb 18, 2025
@ThePooba ThePooba merged commit 2bdb202 into Monkestation:master Feb 18, 2025
32 checks passed
github-actions bot added a commit that referenced this pull request Feb 18, 2025
@LikeLakers2 LikeLakers2 deleted the project/bugfix/monke-5520-bitrunning-chasms branch February 18, 2025 17:47
@LikeLakers2 LikeLakers2 added the Fix fix da bug label Feb 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix fix da bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

You can pull pulse weaponry out of bitrunning domain
3 participants