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

Makes latejoins wake up from cryostasis instead of arriving on a shuttle #2466

Merged
merged 1 commit into from
Jul 28, 2023

Conversation

covertcorvid
Copy link
Member

About The Pull Request

It makes latejoins wake up from cryostasis instead of arriving on a shuttle

Why It's Good For The Game

Thematic

Testing Photographs and Procedure

Screenshots&Videos

cryo

Changelog

🆑
del: Removed arrivals shuttles
tweak: Late joiners will now spawn near the cryostasis pods
/:cl:

@covertcorvid covertcorvid added the Mass Map Change This PR will likely change two or more maps in order to implement its primary feature. label Jul 1, 2023
@covertcorvid covertcorvid requested a review from a team as a code owner July 1, 2023 16:20
@SerynEngi
Copy link
Contributor

Might I ask that you not change the Galactica, Hammerhead, and Aetherwhisp and have me make those changes instead on those maps to avoid mapping conflicts with already open PRs? Particularly as the Galactica and Aetherwhisp fixes in particular fix critical flaws in the maps.

@covertcorvid
Copy link
Member Author

Don't even know if we want this, but even if we do it's low priority and your PRs should be done first

@Bokkiewokkie Bokkiewokkie added Removal Low Priority Not urgent or easier for new coders to fix. labels Jul 4, 2023
Copy link
Contributor

@Bokkiewokkie Bokkiewokkie left a comment

Choose a reason for hiding this comment

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

This should also update the automated announcement system message to state someone has awoken from cryostasis

Copy link
Contributor

@Kmc2000 Kmc2000 left a comment

Choose a reason for hiding this comment

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

Love this.

@Bokkiewokkie Bokkiewokkie merged commit fdfe685 into BeeStation:master Jul 28, 2023
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Active Test Merge Low Priority Not urgent or easier for new coders to fix. Mapping Mass Map Change This PR will likely change two or more maps in order to implement its primary feature. Removal
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants