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

"Return to lobby if preferences unavailable" Revengence #19381

Merged
merged 3 commits into from Jul 20, 2016

Conversation

Incoming5643
Copy link
Contributor

@Incoming5643 Incoming5643 commented Jul 18, 2016

A remake of #4308

🆑
add: You now have the ability to automatically "back out" of a round if you miss your chance at all the jobs you have set to low or higher. This is found on the toggle at the bottom of the jobs window
add: Due to a quirk of game mode code this option will fail to work if you've been selected as an antag (what a tragedy!) In this situation, you'll be assigned a random job.
/:cl:

…doesn't qualify for any jobs they have set. Previously you could only have it set to play a random job (risky and potentially unfun) or play assistant (greytide).
@optimumtact optimumtact added the Feature Exposes new bugs in interesting ways label Jul 18, 2016
@MrStonedOne
Copy link
Member

MrStonedOne commented Jul 18, 2016

experiment: Whatever your previous selection for this field was has been lost, so be sure to update it.

I'll pr an update to your pr that will convert their old preferences to the new system.

@RemieRichards
Copy link
Contributor

Yeah that's unacceptable with the update system we have.

@Incoming5643
Copy link
Contributor Author

felt wrong to burn a version on something so trivial

@RemieRichards
Copy link
Contributor

It's far worse to burn everyone's preference settings.

Makes old savefiles properly work in new system.
@AlexCrimson
Copy link

My hero!

...i still think we need a policy to stop the suiciders though.

@optimumtact
Copy link
Member

Test merge please

@Incoming5643
Copy link
Contributor Author

There might be strange effects of doing a test merge that updates the save file version.

Specifically afterward when the save file is now higher than the max.

@optimumtact
Copy link
Member

Ok

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Exposes new bugs in interesting ways
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants