Skip to content

Add JEI ghost ingredient handling for filters and crafting slots Fixing #213#251

Merged
Ellpeck merged 1 commit intoEllpeck:mainfrom
KatCodesMods:feat/213
Jan 17, 2026
Merged

Add JEI ghost ingredient handling for filters and crafting slots Fixing #213#251
Ellpeck merged 1 commit intoEllpeck:mainfrom
KatCodesMods:feat/213

Conversation

@GaeaKat
Copy link
Copy Markdown
Contributor

@GaeaKat GaeaKat commented Oct 22, 2025

Description

This pull request introduces support for JEI ghost ingredients in filters and crafting slots. It includes:

  • New classes: JEIFilterGhostIngredients and JEICraftingTerminalGhostIngredients to handle ghost ingredient features.
  • Implementation of PacketFilterSlot for server-side updates of filter slots.
  • Update to FilterSlot with a public slotClick method to facilitate packet communication.
  • Registration of JEI ghost ingredient handlers in the JEIPrettyPipesPlugin.

…s fixing Ellpeck#213

- Introduced `JEIFilterGhostIngredients` and `JEICraftingTerminalGhostIngredients` to support ghost ingredients in JEI.
- Added `PacketFilterSlot` to handle server-side updates for filter slots.
- Updated `FilterSlot` with a public `slotClick` method to support packet communication.
- Registered JEI ghost ingredient handlers in `JEIPrettyPipesPlugin`.
@Ellpeck Ellpeck merged commit 6f584b0 into Ellpeck:main Jan 17, 2026
1 check passed
@Ellpeck
Copy link
Copy Markdown
Owner

Ellpeck commented Jan 17, 2026

Hm, it turns out I was a bit too quick to merge these without testing they work. This may be in relation to #250 not having been merged with, but this PR and the non-JEI one (#252) have a number of issues:

Sorry that I so hastily merged these without checking if it all works, I'd assumed these were thoroughly tested because you shared all the videos & screenshots in the Discord server. Again, maybe this is just because #250 has not been merged yet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants