Skip to content

Conversation

@kellygrillojr
Copy link

The OSRS Loot Tracker Plugin connects to a discord bot that I created. It allows the users to sign into their discord account from the plugin and access the severs in which the bot is installed. They then choose what channels they want the plugin to send their loot to. If a drop is detected over the value of the setting they select, a message is sent to those channels.

image

The discord bot then logs it to osrsloottracker.com website so they can share/memorialize their drops with the discord server.

image

@runelite-github-app
Copy link

runelite-github-app bot commented Dec 13, 2025

@runelite-github-app
Copy link

runelite-github-app bot commented Dec 13, 2025

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

@Felanbird
Copy link
Contributor

Could you not just send the loot to that website, with Dink

@kellygrillojr
Copy link
Author

You could, the bot currently scans the channels and logs it that way. But I feel that this is easier to setup/maintain.

@riktenx
Copy link
Member

riktenx commented Dec 15, 2025

Not a full review but you can't directly use HttpUrlConnection - you need to @Inject runelite's OkHttpClient and use that.

@riktenx riktenx added the waiting for author waiting for the pr author to make changes or respond to questions label Dec 15, 2025
@raiyni
Copy link
Member

raiyni commented Dec 15, 2025

Also, is there a specific reason you need to open an OAUTH flow to get Identify Guilds when you could use the DiscordService supplied by runelite?

@kellygrillojr
Copy link
Author

Not a full review but you can't directly use HttpUrlConnection - you need to @Inject runelite's OkHttpClient and use that.

Okay thanks! I'll make this change tonight

@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 Dec 15, 2025
@kellygrillojr
Copy link
Author

Also, is there a specific reason you need to open an OAUTH flow to get Identify Guilds when you could use the DiscordService supplied by runelite?

I didn't know runelite supplied this. I'll look into it. Thanks!

@riktenx riktenx added the waiting for author waiting for the pr author to make changes or respond to questions label Dec 15, 2025
@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 Dec 15, 2025
@kellygrillojr
Copy link
Author

If I understand correctly the Discord service provided by runelite is just for rich presence. My plugin is using its own OAuth flow because of the need to communicate directly with my backend to know which of the users servers have the bot installed.

@riktenx
Copy link
Member

riktenx commented Dec 20, 2025

do not sleep the shared executor service

@riktenx riktenx added the waiting for author waiting for the pr author to make changes or respond to questions label Dec 20, 2025
…dule to get the logged in users RSN as per Riktenx's comment.
@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 Dec 21, 2025
@kellygrillojr
Copy link
Author

kellygrillojr commented Dec 21, 2025

@riktenx the plugin is no longer sleeping the executor service... Thanks!

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.

4 participants