Skip to content

Comments

fix: submenu interaction#1565

Merged
chsami merged 1 commit intomainfrom
development
Oct 14, 2025
Merged

fix: submenu interaction#1565
chsami merged 1 commit intomainfrom
development

Conversation

@chsami
Copy link
Owner

@chsami chsami commented Oct 14, 2025

fix: submenu interaction

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 14, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

  • Bumps microbot.version in runelite-client/pom.xml from 2.0.24 to 2.0.25.
  • Modifies Rs2Inventory.invokeMenu: when the requested action is not in the item widget’s actions, it no longer logs an error or returns early. If a sub-action mapping exists, it computes the main menu index, updates the identifier accordingly, and clears the target. If no sub-action mapping exists, the method continues with identifier -1 and target set to the item name.

Possibly related PRs

✨ Finishing touches
  • 📝 Generate docstrings

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 1abefa8 and 5908d72.

📒 Files selected for processing (2)
  • runelite-client/pom.xml (1 hunks)
  • runelite-client/src/main/java/net/runelite/client/plugins/microbot/util/inventory/Rs2Inventory.java (1 hunks)

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@chsami chsami merged commit 8effa51 into main Oct 14, 2025
2 of 3 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.

1 participant