Skip to content

Call EntityCollideWithEntityEvent for Sulfur Cube pushed by Player - #14157

Merged
Doc94 merged 1 commit into
PaperMC:ver/26.2from
Doc94:feature/sulfur-cube-push-by-player
Sep 20, 2026
Merged

Doc94 merged 1 commit into
PaperMC:ver/26.2from
Doc94:feature/sulfur-cube-push-by-player

Conversation

@Doc94

@Doc94 Doc94 commented Aug 7, 2026

Copy link
Copy Markdown
Member

With the addition of 9119d1d
This PR include the case for Sulfur Cube with item, this follow another logic for "playerTouch" rather than push.

also apply the JSpecify changes.

@Doc94
Doc94 requested a review from a team as a code owner August 7, 2026 15:02
@github-project-automation github-project-automation Bot moved this to Awaiting review in Paper PR Queue Aug 7, 2026
@Doc94
Doc94 force-pushed the feature/sulfur-cube-push-by-player branch from 21662c3 to 6bdfebf Compare August 7, 2026 16:00
iambibi added a commit to ServerOpenMC/PluginV2 that referenced this pull request Aug 11, 2026
@CatTeaA

CatTeaA commented Aug 16, 2026

Copy link
Copy Markdown

After several days of testing and use, it's running well. Does this PR have a chance to be merged before 26.3 arrives?

@papermc-projects papermc-projects Bot moved this from Awaiting review to Delayed approval in Paper PR Queue Aug 30, 2026
@papermc-projects papermc-projects Bot moved this from Delayed approval to Full approval in Paper PR Queue Sep 14, 2026
@Doc94
Doc94 changed the base branch from main to ver/26.2 September 20, 2026 02:25
@Doc94
Doc94 force-pushed the feature/sulfur-cube-push-by-player branch from 6bdfebf to f69a41e Compare September 20, 2026 02:26
@Doc94
Doc94 merged commit c646faf into PaperMC:ver/26.2 Sep 20, 2026
4 checks passed
@github-project-automation github-project-automation Bot moved this from Full approval to Merged in Paper PR Queue Sep 20, 2026
@Doc94
Doc94 deleted the feature/sulfur-cube-push-by-player branch September 20, 2026 02:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Merged

Development

Successfully merging this pull request may close these issues.

3 participants