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

[BUG] Drop is deemed valuable, when item is not valuable due to display price type conflicts. #1

Closed
Anderzenn opened this issue Nov 3, 2024 · 0 comments
Labels
bug Something isn't working

Comments

@Anderzenn
Copy link
Owner

Describe the bug
When an item is dropped that has a default value higher than that of the default valuable drop "coins" message, and display price type is set to High Alch. The chat message will write "Valuable drop: item (HA: No Value)".

To Reproduce
Set display price type to high alch, get a drop that has no high alch value but a grand exchange/default value above the set threshold and watch the chat message.

Expected behavior
Instead of writing anything at all, it should simply prevent the message from appearing in the first place.

@Anderzenn Anderzenn added the bug Something isn't working label Nov 3, 2024
Anderzenn added a commit that referenced this issue Nov 3, 2024
- Fixed bug relating to issue #1
Anderzenn added a commit to Anderzenn/plugin-hub that referenced this issue Nov 3, 2024
- Fixed misspelled link in plugin properties
- Fixed bug relating to Anderzenn/valuable-drop-prices#1
- Removed unused code from config
- Updated readme & set gradle settings project name to plugins name.
tylerwgrass pushed a commit to runelite/plugin-hub that referenced this issue Nov 4, 2024
* Create valuable-drop-prices

Simple plugin that replaces the default "coins" value with HA and GE prices to the valuable drop notifications.

* Update valuable-drop-prices

Updated to be using the correct commit

* Update valuable-drop-prices

- Fixed misspelled link in plugin properties
- Fixed bug relating to Anderzenn/valuable-drop-prices#1
- Removed unused code from config
- Updated readme & set gradle settings project name to plugins name.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Comitted
Development

No branches or pull requests

1 participant