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.

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

Search all slots with /searchinv #218

Merged
merged 2 commits into from
Jul 7, 2024
Merged

Search all slots with /searchinv #218

merged 2 commits into from
Jul 7, 2024

Conversation

Jikoo
Copy link
Owner

@Jikoo Jikoo commented Jul 7, 2024

This has been unresolved for way, way too long, so we're going with the quick fix instead of rewriting search to be better.

Closes #14

Jikoo added 2 commits July 7, 2024 09:47
Containment check is done against storage contents only; have to implement our own check.
Update to modern NamespacedKey/Enchantment usage
@Jikoo Jikoo enabled auto-merge (squash) July 7, 2024 14:03
@Jikoo Jikoo merged commit 1cfa9bd into master Jul 7, 2024
2 checks passed
@Jikoo Jikoo deleted the dev/search-all-slots branch July 7, 2024 14:06
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.

/searchinv doesn't include armor or offhand slots
1 participant