Skip to content

[1.21.4] Re-introduce FluidStack item handlers #10411

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.

Sign up for GitHub

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

Jump to bottom
Draft
Jonathing wants to merge 1 commit into
base: 1.21.4
Choose a base branch
from
Draft

[1.21.4] Re-introduce FluidStack item handlers #10411

Jonathing wants to merge 1 commit into from

Conversation

@Jonathing
Copy link
Member

@Jonathing Jonathing commented Feb 12, 2025

Name entails. This is the same system used pre-1.20.6. The only differences are that data is stored using a data component and I cleaned up the formatting of the relevant files a little bit.

@autoforge autoforge bot added Triage This request requires the active attention of the Triage Team. Requires labelling or reviews. LTS This issue/PR is related to the current LTS version. 1.21 labels Feb 12, 2025
@Jonathing
Copy link
Member Author

Jonathing commented Feb 12, 2025

Drafted until game tests. Let me know if there's anything I missed.

@Jonathing Jonathing added Regression This request demonstrates something missing in a newer version that was present in a prior version. Capabilities This request relates to the capability system. Lots of care is required, so progress may be slow. and removed LTS This issue/PR is related to the current LTS version. labels Feb 12, 2025
@Jonathing Jonathing requested a review from LexManos February 12, 2025 03:11
@autoforge autoforge bot added the Needs Rebase This PR requires a rebase to implement upstream changes (usually to fix merge conflicts). label Mar 26, 2025
@autoforge
Copy link

autoforge bot commented Mar 26, 2025

@Jonathing, this pull request has conflicts, please resolve them for this PR to move forward.

@Jonathing Jonathing changed the base branch from 1.21.x to 1.21.4 March 27, 2025 02:18
@autoforge autoforge bot removed the Needs Rebase This PR requires a rebase to implement upstream changes (usually to fix merge conflicts). label Jul 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@LexManos LexManos Awaiting requested review from LexManos

Assignees

No one assigned

Labels

1.21 Capabilities This request relates to the capability system. Lots of care is required, so progress may be slow. Regression This request demonstrates something missing in a newer version that was present in a prior version. Triage This request requires the active attention of the Triage Team. Requires labelling or reviews.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[1.21.4] FluidUtil.getFluidHandler missing implementation

1 participant

@Jonathing