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

LADX: Fix hints crash #2050

Merged
merged 1 commit into from
Jul 29, 2023
Merged

Conversation

zig-for
Copy link
Collaborator

@zig-for zig-for commented Jul 29, 2023

Probably fixes the crash - TIL unplaced items are normal.

@ThePhar ThePhar added the is: bug/fix Issues that are reporting bugs or pull requests that are fixing bugs. label Jul 29, 2023
@ThePhar
Copy link
Member

ThePhar commented Jul 29, 2023

If a game has unplaced items it's technically a bug for the world that created extra items.

@el-u
Copy link
Collaborator

el-u commented Jul 29, 2023

What's going on here? It looks like the PR consists of a 1-line change to work around a crash and then 99 other lines to introduce a (completely unrelated?) new option.

@el-u
Copy link
Collaborator

el-u commented Jul 29, 2023

If a game has unplaced items it's technically a bug for the world that created extra items.

The log posted in https://discord.com/channels/731205301247803413/1134521806099853352 reports neither unplaced items nor unfilled locations.

@zig-for
Copy link
Collaborator Author

zig-for commented Jul 29, 2023

I forked off of a feature branch by accident

@zig-for
Copy link
Collaborator Author

zig-for commented Jul 29, 2023

Fixed

@Berserker66 Berserker66 merged commit 6ad33bb into ArchipelagoMW:main Jul 29, 2023
FlySniper pushed a commit to FlySniper/Archipelago that referenced this pull request Nov 14, 2023
kl3cks7r pushed a commit to kl3cks7r/Archipelago that referenced this pull request Dec 15, 2023
Jouramie pushed a commit to Jouramie/Archipelago that referenced this pull request Feb 28, 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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants