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

Lingo: Fix broken good item in panelsanity #3249

Merged
merged 1 commit into from
May 2, 2024

Conversation

hatkirby
Copy link
Collaborator

@hatkirby hatkirby commented May 2, 2024

What is this fixing or adding?

There was an issue where the combination of door shuffle, panelsanity, no early color hallways, and being in a multi player world would break generation. This was because of the code that generated an early good item. The code is not supposed to run under panelsanity, but the conditional was not correct. This fixes that.

How was this tested?

Pytest, and running a test generation under the specific error case from above.

If this makes graphical changes, please attach screenshots.

@hatkirby hatkirby requested a review from NewSoupVi May 2, 2024 11:43
@github-actions github-actions bot added the waiting-on: peer-review Issue/PR has not been reviewed by enough people yet. label May 2, 2024
@hatkirby hatkirby added the is: bug/fix Issues that are reporting bugs or pull requests that are fixing bugs. label May 2, 2024
@NewSoupVi NewSoupVi merged commit 0d586a4 into ArchipelagoMW:main May 2, 2024
16 checks passed
@hatkirby hatkirby deleted the goodluck branch May 2, 2024 16:07
jnschurig pushed a commit to Tranquilite0/Archipelago-SoulBlazer that referenced this pull request Jun 13, 2024
qwint pushed a commit to qwint/Archipelago that referenced this pull request Jun 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
is: bug/fix Issues that are reporting bugs or pull requests that are fixing bugs. waiting-on: peer-review Issue/PR has not been reviewed by enough people yet.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants