Skip to content

Conversation

kevinthegreat1
Copy link
Collaborator

@kevinthegreat1 kevinthegreat1 commented Nov 23, 2023

Add Wither & Blood Door Highlight!

Known bugs:
Some key pickups by other players are not detected. (When the player entity is not loaded or when the key is obtained in a room not yet visited by the client's player.)
Wither door in the fairy room always remains locked since no key can ever be picked up in the fairy room.

@kevinthegreat1 kevinthegreat1 added new feature This issue or PR is a new feature reviews needed This PR needs reviews labels Nov 23, 2023
@kevinthegreat1 kevinthegreat1 added this to the 1.16 milestone Nov 23, 2023
Copy link
Collaborator

@AzureAaron AzureAaron left a comment

Choose a reason for hiding this comment

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

Aside from the bugs stated this is it, otherwise its fine to me

@AzureAaron AzureAaron added changes requested This PR need changes and removed reviews needed This PR needs reviews labels Nov 23, 2023
@kevinthegreat1 kevinthegreat1 added reviews needed This PR needs reviews and removed changes requested This PR need changes labels Nov 23, 2023
@AzureAaron AzureAaron added merge me please Pull requests that are ready to merge and removed reviews needed This PR needs reviews labels Nov 23, 2023
@pigerstreet
Copy link

zzz

@pigerstreet
Copy link

Seems to work fine, however sometimes when I get the key the highlight stays red.

@pigerstreet
Copy link

Also when I enter the fairy room with a key it always seems to be red

@kevinthegreat1 kevinthegreat1 merged commit 6eb9559 into SkyblockerMod:master Nov 24, 2023
@kevinthegreat1 kevinthegreat1 deleted the wither-blood-door-esp branch November 24, 2023 20:01
@AzureAaron AzureAaron removed the merge me please Pull requests that are ready to merge label Nov 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new feature This issue or PR is a new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants