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.

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

alloy smelter mixin injection failure #1

Closed
justliliandev opened this issue Sep 21, 2023 · 3 comments · Fixed by #2
Closed

alloy smelter mixin injection failure #1

justliliandev opened this issue Sep 21, 2023 · 3 comments · Fixed by #2
Labels
state: completed type: feature/enhancement

Comments

@justliliandev
Copy link
Contributor

Version

0.3.0

Crash Report

https://pste.me/@R1eOG8SKEaA

Log

No response

Additional Context

Yes

Modifications

it's a bigger modpack, however, would you mind working with us to remove the mixin alltogether, I would much rather expose something in the api then have mixins target non api classes?

Reference: Team-EnderIO/EnderIO#520

Did the crash happen in singleplayer or on a server?

Both
@justliliandev
Copy link
Contributor Author

This PR has been setup to remove the mixin: Team-EnderIO/EnderIO#521

@Rover656
Copy link

The PR above has been merged and released in version 6.0.20-alpha. Thanks :)

@justliliandev justliliandev mentioned this issue Sep 21, 2023
Merged
@rlnt
Copy link
Member

rlnt commented Sep 21, 2023

Understandable, thanks for the API endpoint.

@rlnt rlnt closed this as completed in #2 Sep 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
state: completed type: feature/enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

remove mixin into alloysmelter and use integration instead justliliandev/kubejs-enderio
3 participants
@rlnt @Rover656 @justliliandev