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

Detailed Crafting bugs #173

Closed
anom3 opened this issue Jan 12, 2022 · 8 comments
Closed

Detailed Crafting bugs #173

anom3 opened this issue Jan 12, 2022 · 8 comments
Labels
bug

Comments

@anom3
Copy link

anom3 commented Jan 12, 2022

Describe the bug
There is no JEI integration with the Detailed Auto-Crafter and multi-path recipes require a specific path in the Detailed Auto-Crafter.

To Reproduce [Optional]
Steps to reproduce the behavior:

  1. Open Detailed Auto-Crafter GUI
  2. Open JEI tab for Detailed Crafting
  3. See that there is no plus.
  4. Insert Detailed Crafting recipe for Detailed Crafter
  5. See that you need a specific sigil (witchcraft for me)

Expected behavior
3. There is a plus.
5. Any sigil works.
@anom3 anom3 added the bug label Jan 12, 2022
@Da-Technomancer
Copy link
Member

Which recipe were you trying that it required witchcraft and you expected something else?

@anom3
Copy link
Author

anom3 commented Jan 12, 2022

Actually I think you misunderstood. With the Detailed Auto-Crafter SPECIFICALLY, it required witchcraft to make Detailed Crafters with the cheap recipe. I expected it to work with any path.

@anom3
Copy link
Author

anom3 commented Jan 12, 2022

It works with any path on the manual Detailed Crafter.

@anom3
Copy link
Author

anom3 commented Jan 12, 2022

(I think) I can test

@anom3
Copy link
Author

anom3 commented Jan 12, 2022

aaand yup it works with any path manually.

@anom3
Copy link
Author

anom3 commented Jan 12, 2022 •
edited
Loading

the recipe
image

@anom3
Copy link
Author

anom3 commented Jan 12, 2022 •
edited
Loading

I think there is a relatively easy way to fix the auto specific sigil requirement. Making multi path recipes (recipes that are allowed in multiple paths) one recipe internally that specifies multiple allowed paths should work.

Da-Technomancer added a commit that referenced this issue Jan 12, 2022
@Da-Technomancer
Fixed detailed autocrafter JEI issue, multi-path issues
f8d3b09 
*Fixed the detailed autocrafter requiring a specific path for recipes with multiple valid paths
+Added JEI support to the detailed autocrafter. Known issue: The recipe configured with JEI won't be visible in the interface at all unless the correct sigil is installed.
Closed #173
@Da-Technomancer
Copy link
Member

Created a new issue in the process of fixing this: #174

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Da-Technomancer @anom3