Skip to content

feat: initial runealytics plugin#10806

Open
mrtms wants to merge 6 commits intorunelite:masterfrom
Runescape-Tracking:feat/runelite-plugin-hub
Open

feat: initial runealytics plugin#10806
mrtms wants to merge 6 commits intorunelite:masterfrom
Runescape-Tracking:feat/runelite-plugin-hub

Conversation

@mrtms
Copy link
Copy Markdown

@mrtms mrtms commented Mar 1, 2026

No description provided.

@runelite-github-app
Copy link
Copy Markdown

runelite-github-app Bot commented Mar 1, 2026

Includes changes by non-author

New plugin runealytics: https://github.com/Runescape-Tracking/RuneAlytics-Plugin/tree/04707a5031dc19f266798494ffde3ba02244f8db

@runelite-github-app
Copy link
Copy Markdown

runelite-github-app Bot commented Mar 1, 2026

This plugin requires a review from a Plugin Hub maintainer. The reviewer will request any additional changes if needed.


Internal use only: Reviewer details Maintainer details

@raiyni
Copy link
Copy Markdown
Member

raiyni commented Mar 1, 2026

It looks like there is a difference between what you say this plug-in does and what it actually does. Eg what is this match making stuff?

@mrtms
Copy link
Copy Markdown
Author

mrtms commented Mar 1, 2026

It looks like there is a difference between what you say this plug-in does and what it actually does. Eg what is this match making stuff?

It looks like there might be some confusion about what the plugin actually does.

This isn’t just a loot tracking & banking syncing plugin. It’s a multi-feature plugin.

It includes:

Loot tracking: When you kill an NPC and get a drop, the item is automatically tracked on your account and synced to our website.

Banking Sync - Syncs your OSRS bank to our website

Matchmaking (still being tested): We’re building a PvP matchmaking system that pairs you with players of similar skill level — kind of like how Gamebattles/MLG used to work. It’s still in testing and we’re ironing out the kinks.

Account verification: There’s also a verification system that links your in-game account to our website (runealytics.com).

@cshunton
Copy link
Copy Markdown

cshunton commented Mar 1, 2026

It looks like there is a difference between what you say this plug-in does and what it actually does. Eg what is this match making stuff?

Our next major update will include the full match making system but we have a little more testing to do with it before we make it live. Our goal is to allow players to select a specific type of PvP match that they enjoy to do, and learn how to perfect it and play against other players around the same skill as them. Once our system is complete, there will be an elo system that will help match skill set

@raiyni
Copy link
Copy Markdown
Member

raiyni commented Mar 1, 2026

Why do you need bank contents and IGN, human please reply

@cshunton
Copy link
Copy Markdown

cshunton commented Mar 1, 2026

Why do you need bank contents and IGN, human please reply

The bank system is used to show off your loot, by default this is always off (not tracked and must be enabled by the user if they want to track it. On the site it is also defaulted to private so if the user select to sync banks then it must be also enabled on the site if they want the public to view it.

It goes hand in hand with out loot tracking system, it allows the website to track your GP gains if the user wants to

Screenshot 2026-03-01 054333

@mhollink
Copy link
Copy Markdown
Contributor

mhollink commented Mar 1, 2026

Question from a bystander; doesn't this require a warning like wom-utils and such?

@cshunton
Copy link
Copy Markdown

cshunton commented Mar 1, 2026

Question from a bystander; doesn't this require a warning like wom-utils and such?

We can definitely add that

@cshunton
Copy link
Copy Markdown

cshunton commented Mar 5, 2026

Why do you need bank contents and IGN, human please reply

Was i able to answer your question?

Copy link
Copy Markdown

@cshunton cshunton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is an example of the data the plugin uses to show off your drops/XP.

image

@mhollink

@mhollink
Copy link
Copy Markdown
Contributor

.

What a passive agressieve way to ping the reviewers who are currently swamped in work 😒

@mrtms
Copy link
Copy Markdown
Author

mrtms commented Mar 16, 2026

.

What a passive agressieve way to ping the reviewers who are currently swamped in work 😒

Chill out. He's new to GitHub and being a dev; he was on his phone and accidentally commented a .. It was edited once he noticed it, you were just quick with a response.

Comment thread plugins/runealytics Outdated
@LlemonDuck LlemonDuck added the waiting for author waiting for the pr author to make changes or respond to questions label Apr 6, 2026
Updated to suggested `warning` output text.

Co-authored-by: Rhea <napkinorton@gmail.com>
@runelite-github-app runelite-github-app Bot removed the waiting for author waiting for the pr author to make changes or respond to questions label Apr 7, 2026
@mrtms mrtms requested a review from LlemonDuck April 7, 2026 19:53
Comment thread plugins/runealytics Outdated
@LlemonDuck
Copy link
Copy Markdown
Contributor

typo in my own suggestion, sorry

Update `warning` message

Co-authored-by: Rhea <napkinorton@gmail.com>
@mrtms
Copy link
Copy Markdown
Author

mrtms commented Apr 7, 2026

typo in my own suggestion, sorry

All good! Committed the update!

@mrtms mrtms requested a review from LlemonDuck April 7, 2026 22:08
@LlemonDuck
Copy link
Copy Markdown
Contributor

do you intend for the default url to be a testing url? https://github.com/Runescape-Tracking/RuneAlytics-Plugin/blob/f4e2657ad857ac785eb4c79083133ebb4bfefb42/src/main/java/com/runealytics/RunealyticsConfig.java#L67

@LlemonDuck LlemonDuck self-assigned this Apr 7, 2026
@LlemonDuck LlemonDuck added the waiting for author waiting for the pr author to make changes or respond to questions label Apr 7, 2026
Updates link from `testing.runealytics.com` to `runealytics.com`
@runelite-github-app runelite-github-app Bot removed the waiting for author waiting for the pr author to make changes or respond to questions label Apr 7, 2026
@mrtms
Copy link
Copy Markdown
Author

mrtms commented Apr 7, 2026

do you intend for the default url to be a testing url? https://github.com/Runescape-Tracking/RuneAlytics-Plugin/blob/f4e2657ad857ac785eb4c79083133ebb4bfefb42/src/main/java/com/runealytics/RunealyticsConfig.java#L67

Nice catch and probably on the safer side if we do the main server rather than the testing one since that's where we deploy testing code to. Thanks for that - appreciate the thorough code review!

@000744210
Copy link
Copy Markdown
Contributor

I notice you could make your bank stats public. Does that go against this?

Hiscores for PBs and other information not on the hiscores: Same as the GE Tracker, the data could easily be spoofed, rendering it useless

https://github.com/runelite/runelite/wiki/Rejected-or-Rolled-Back-Features

@LlemonDuck
Copy link
Copy Markdown
Contributor

remove runelite-plugin.properties from /src/main/resources

gitignore /out while you're at it

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.

7 participants