Skip to content

Conversation

@ARTARNA
Copy link
Contributor

@ARTARNA ARTARNA commented Dec 13, 2025

Fixes #177

@LlemonDuck
Copy link
Owner

this won't work if the player clicks and needs to wait multiple ticks walking to the chest. also it's failure-prone to clicking, cancelling the action, and doing anything else that modifies the inventory.

something akin to checking for the menu action and then looking for proximity+location would be better, like what I did in the core farming plugin, but it's fairly involved https://github.com/runelite/runelite/blob/f621f01f9d6c8e5f4d01faa388df0efbd9cfc6c3/runelite-client/src/main/java/net/runelite/client/plugins/timetracking/farming/CompostTracker.java

@ARTARNA
Copy link
Contributor Author

ARTARNA commented Dec 16, 2025

this won't work if the player clicks and needs to wait multiple ticks walking to the chest. also it's failure-prone to clicking, cancelling the action, and doing anything else that modifies the inventory.

something akin to checking for the menu action and then looking for proximity+location would be better, like what I did in the core farming plugin, but it's fairly involved https://github.com/runelite/runelite/blob/f621f01f9d6c8e5f4d01faa388df0efbd9cfc6c3/runelite-client/src/main/java/net/runelite/client/plugins/timetracking/farming/CompostTracker.java

Ahh I see, makes sense. I will have a look and a think. Thanks for the poke in the right direction

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.

Cargo hold tracking does not detect items added via quick-deposit menu option

2 participants