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

Improvement: Estimated Item Value Mithril Infusion #2990

Conversation

ItsEmpa
Copy link
Contributor

@ItsEmpa ItsEmpa commented Nov 26, 2024

What

This PR adds support for adding mithril infused minions to estimated item value and chest value. Also removes most .toInternalName() calls in estimated item value to be variables in the class, and adds .toInternalNames() functions for converting a list/set of string to list/set of internal name

Changelog Improvements

  • Added Mithril Infusion to Estimated Item Value. - Empa

@hannibal002 hannibal002 added this to the Version 0.29 milestone Nov 26, 2024
@github-actions github-actions bot added the Merge Conflicts There are open merge conflicts with the beta branch. label Dec 25, 2024
Copy link

This pull request has conflicts with the base branch "beta". Please resolve those so we can test out your changes.

…d-item-value-mithril-infusion

# Conflicts:
#	src/main/java/at/hannibal2/skyhanni/features/misc/items/EstimatedItemValueCalculator.kt
Copy link

Conflicts have been resolved! 🎉

@github-actions github-actions bot removed the Merge Conflicts There are open merge conflicts with the beta branch. label Dec 25, 2024
@hannibal002 hannibal002 merged commit ffc6d1d into hannibal002:beta Dec 25, 2024
6 checks passed
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.

3 participants