Skip to content

Allow Bottomless Pouch in Focus Pouch slot #62

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.

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

Merged
merged 2 commits into from
Jul 23, 2025
Merged

Conversation

koolkrafter5
Copy link

Slot is added by Salis Arcana. Does not register the slot itself if the mod is missing, since the regular focus pouch wouldn't be able to be worn in it and that would be weird. It also probably wouldn't even be functional without Salis since it adds a bunch of mixins for Thaum to use BE's slots.

image image

"Focus Pouch" not being green is an issue with Salis that I will make another PR for.
Foci are pulled from both as expected.

…y Salis Arcana. Does not register the slot itself if the mod is missing, since the regular focus pouch wouldn't be able to be worn in it.
@koolkrafter5 koolkrafter5 requested a review from a team July 23, 2025 05:20
@Nikolay-Sitnikov Nikolay-Sitnikov merged commit 7b02309 into master Jul 23, 2025
1 check passed
@Nikolay-Sitnikov Nikolay-Sitnikov deleted the pouch branch July 23, 2025 05:42
rndmorris pushed a commit to rndmorris/Salis-Arcana that referenced this pull request Jul 23, 2025
**What kind of change does this PR introduce?** (Bug fix, feature, docs
update, ...)
Bug fix

**What is the current behavior?** (You can also link to an open issue
here)
<img width="972" height="265" alt="image"
src="https://github.com/user-attachments/assets/b9acec4e-11ef-4aab-bce9-ab21b89d98ff"
/>
GTNewHorizons/ThaumicTinkerer#62

**What is the new behavior (if this is a feature change)?**
That text is now green

**Does this PR introduce a breaking change?**
No

**Other information**:
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants