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

Option to stop including items in Equipment Sets in the Sell/Destroy list #181

Closed
Vettic opened this issue Mar 12, 2023 · 0 comments · Fixed by #188
Closed

Option to stop including items in Equipment Sets in the Sell/Destroy list #181

Vettic opened this issue Mar 12, 2023 · 0 comments · Fixed by #188

Comments

@Vettic
Copy link

Vettic commented Mar 12, 2023

Dejunk will include items from equipment sets in its Dejunk window and will sell/destroy them unless each item is explicitly excluded.
This should be considered undesirable functionality as once that item is replaced in an equipment set it will continue to not be present in the Dejunk window unless removed from the exclusion list, if the exclusion list contains many entries the user will find this a hassle, If the item is used on many alts it will be excluded on all of them if present in the global list.

There should be a checkbox in the options titled "Include equipment sets", defaulting to unchecked to avoid loss of items for new users.

When unchecked, the sell/destroy list should dynamically ignore items that are present in the current characters equipment sets when populating its list.

When checked, the items can be included if they are not part of any relevant exclusion lists, and the sell/destroy list can function as it currently does.

@moody moody mentioned this issue Dec 25, 2023
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 a pull request may close this issue.

1 participant