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

DOOM II: Fix sector 95 assignment in DOOM II MAP17 to correctly flag … #3705

Merged
merged 1 commit into from
Sep 17, 2024

Conversation

sgrunt
Copy link
Contributor

@sgrunt sgrunt commented Jul 28, 2024

…the BFG9000 location as in the Yellow Key area

What is this fixing or adding?

Related to APDOOM fix Daivuk/apdoom#21, this fixes a logic error in the DOOM II apworld.

How was this tested?

Double checking locally after making the change that granting access to MAP17 does not flag the BFG9000 location as accessible - only after getting access the MAP17 yellow key area (requiring both the red and yellow keys) is the item considered accessible.

If this makes graphical changes, please attach screenshots.

N/A

…the BFG9000 location as in the Yellow Key area
Copy link
Contributor

@KScl KScl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One-line fix to an auto-generated file that places the relevant item in the correct region. The related PR to APDoom changes ap_gen_tool's sources to generate the file with this fix, ensuring it is maintained.

@Exempt-Medic Exempt-Medic added the waiting-on: world-maintainer Issue/PR is waiting for feedback or approval by the maintainer of a world. label Jul 28, 2024
@Exempt-Medic Exempt-Medic requested a review from Daivuk July 28, 2024 20:25
@Exempt-Medic Exempt-Medic added the is: bug/fix Issues that are reporting bugs or pull requests that are fixing bugs. label Jul 28, 2024
@Exempt-Medic Exempt-Medic added the affects: release/blocker Issues/PRs that must be addressed before next official release. label Sep 17, 2024
@Exempt-Medic
Copy link
Member

@Daivuk Just a reminder to look at this

@Daivuk
Copy link
Collaborator

Daivuk commented Sep 17, 2024

@KScl confirmed the change was also made in the tool. So this should be good to go! Approved.

@Exempt-Medic Exempt-Medic added waiting-on: core-review Issue/PR has been peer-reviewed and is ready to be merged or needs input from a core maintainer. and removed waiting-on: world-maintainer Issue/PR is waiting for feedback or approval by the maintainer of a world. waiting-on: peer-review Issue/PR has not been reviewed by enough people yet. labels Sep 17, 2024
@NewSoupVi NewSoupVi merged commit debb936 into ArchipelagoMW:main Sep 17, 2024
3 checks passed
AustinSumigray pushed a commit to AustinSumigray/Archipelago that referenced this pull request Jan 4, 2025
…the BFG9000 location as in the Yellow Key area (ArchipelagoMW#3705)

Co-authored-by: sgrunt <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affects: release/blocker Issues/PRs that must be addressed before next official release. is: bug/fix Issues that are reporting bugs or pull requests that are fixing bugs. waiting-on: core-review Issue/PR has been peer-reviewed and is ready to be merged or needs input from a core maintainer.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants