Skip to content

Update bankstanding to latest version#11299

Open
mhollink wants to merge 2 commits intorunelite:masterfrom
mhollink:bankstanding
Open

Update bankstanding to latest version#11299
mhollink wants to merge 2 commits intorunelite:masterfrom
mhollink:bankstanding

Conversation

@mhollink
Copy link
Copy Markdown
Contributor

  • Fixes issue where loading an instance resets the player state to AFK
  • Removes ExpGained events when 0 exp was granted

This new version mainly includes the 2 following user facing changes
- Allow bankstanding experience to be shortent with K and M formats for higher levels
- Add bank display names to nicely translate the enum.name() values

Aside from those changes..;
- Quite refactoring has been done to move code to a 'better' place & improve testablity
- Added some unit tests to improve stability for the of the inner-workings of the plugin
- Updated the readme and added a bounty for bug reports regarding missing/incorrect bank locations.
@runelite-github-app
Copy link
Copy Markdown

@runelite-github-app
Copy link
Copy Markdown

This plugin requires reviews from Plugin Hub reviewers. The reviewer will request any additional changes if needed.


Internal use only: Reviewer details Maintainer details

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant