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

KH2: Docs updates and Excluded Locations Bugfix #3150

Merged
merged 2 commits into from
Apr 18, 2024

Conversation

JaredWeakStrike
Copy link
Collaborator

Please format your title with what portion of the project this pull request is
targeting and what it's changing.

ex. "MyGame4: implement new game" or "Docs: add new guide for customizing MyGame3"

What is this fixing or adding?

Update to the docs to reflect the removal of the qol stuff from the companion mod and moved into the qol mod.
Fixed the bug that caused the generation to crash if someone excluded keyblade slots.
Added the seed name to the randomizer seed so you can see which seed is which in the mod manager
Fixed some grammar in the cor skip logic option description

How was this tested?

ran the tests, generated with a yaml that excluded locations and read the setup guides

If this makes graphical changes, please attach screenshots.

@github-actions github-actions bot added the waiting-on: peer-review Issue/PR has not been reviewed by enough people yet. label Apr 15, 2024
@ThePhar ThePhar added is: bug/fix Issues that are reporting bugs or pull requests that are fixing bugs. is: documentation Improvements or additions to documentation. labels Apr 15, 2024
@Berserker66 Berserker66 merged commit 2704015 into ArchipelagoMW:main Apr 18, 2024
15 checks passed
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. is: documentation Improvements or additions to documentation. 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.

None yet

3 participants