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

Ender eye placement in portal frame #5913

Open
wants to merge 6 commits into
base: minor-next
Choose a base branch
from

Conversation

ShockedPlot7560
Copy link
Member

Introduction

This is an update/improvement of : #4850

Changes

Behavioural changes

  • Add an eye in a frame when a player interacts with it.

Tests

2023-07-18-18-33-33.mp4

@ShockedPlot7560 ShockedPlot7560 added Category: Gameplay Related to Minecraft gameplay experience Type: Enhancement Contributes features or other improvements to PocketMine-MP labels Jul 18, 2023
src/world/sound/EndPortalFillSound.php Outdated Show resolved Hide resolved
src/block/EndPortalFrame.php Show resolved Hide resolved
Joshy3282 added a commit to Joshy3282/PocketMine-MP that referenced this pull request Oct 2, 2024
@dktapps
Copy link
Member

dktapps commented Nov 14, 2024

I stalled on this because I don't like the Eye of Ender being added as a generic item. From reading the code there's no obvious indication that there's more work to be done (e.g. adding projectile functionality).

Unsure if this might cause BC blockers for PM5 if additional functionality were to be implemented.

@dktapps dktapps requested a review from a team as a code owner November 14, 2024 18:02
github-actions[bot]
github-actions bot previously approved these changes Nov 14, 2024
github-actions[bot]
github-actions bot previously approved these changes Nov 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category: Gameplay Related to Minecraft gameplay experience Status: Waiting on Author Type: Enhancement Contributes features or other improvements to PocketMine-MP
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants