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 default random distribution for multiselect settings #2178

Merged
merged 2 commits into from
Apr 22, 2024

Conversation

fenhl
Copy link
Collaborator

@fenhl fenhl commented Feb 15, 2024

Fixes #2118. Tested to ensure this generates spoiler logs with correct values of spawn_positions.

@fenhl fenhl added Type: Bug Something isn't working Status: Needs Review Someone should be looking at it Component: Setting specific to setting(s) labels Feb 15, 2024
@cjohnson57 cjohnson57 merged commit 5b70c64 into OoTRandomizer:Dev Apr 22, 2024
3 checks passed
@fenhl fenhl deleted the random-multiselect branch April 22, 2024 01:18
@fenhl fenhl added this to the next milestone Apr 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Setting specific to setting(s) Status: Needs Review Someone should be looking at it Type: Bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Spawn Position shuffle options with Randomize Main Settings are interpreted as an array of characters
2 participants