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

Fix unpacking vendors not performing area entry #6105

Merged
merged 2 commits into from
Apr 13, 2024

Conversation

Drulikar
Copy link
Contributor

@Drulikar Drulikar commented Apr 7, 2024

About the pull request

This PR fixes unboxing vendors not entering the area to try and connect to the powernet. Note - this will mean all items getting picked up and dropped are now entering/exiting areas correctly. Possible there's some obscure new behavior with this but its unlikely.

Explain why it's good for the game

If you unbox a machine in a powered area, it should be powered.

Testing Photographs and Procedure

Screenshots & Videos
power.mp4

Changelog

🆑 Drathek
fix: Fixed unboxed machines not connecting to the power net
/:cl:

@github-actions github-actions bot added the Fix Fix one bug, make ten more label Apr 7, 2024
@morrowwolf
Copy link
Member

morrowwolf commented Apr 8, 2024

This is the third (fourth?) time someone has put up a PR to fix this. This one will beat the curse I believe.

LGTM

@fira
Copy link
Member

fira commented Apr 8, 2024

an issue i see with this is that if you say pocket an item then put it on the ground repeatedly, it'll repeatedly "Enter" the area without ever exiting it

i think the same change should be made further up in the exit check to stay consistent

@harryob harryob added this pull request to the merge queue Apr 13, 2024
Merged via the queue into cmss13-devs:master with commit bf67b33 Apr 13, 2024
26 checks passed
cm13-github added a commit that referenced this pull request Apr 13, 2024
@Drulikar Drulikar deleted the Fix_Unpacking_Vendors branch April 13, 2024 13:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix Fix one bug, make ten more
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants