Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create lfghelper #7412

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Create lfghelper #7412

wants to merge 4 commits into from

Conversation

Lorkrakt
Copy link

The LFG Helper plugin helps RuneScape players quickly find groups for PVM content by providing a simple, user-friendly interface in the RuneLite sidebar. Players can specify group details like boss name, team size, clanchat, and world, and submit these details to a designated Discord channel via a webhook.

Features:
Simple Form Interface: Enter boss name, team size, clanchat, and more.
Discord Integration: Sends LFG request to a Discord channel with a custom webhook.
Role Support: Sends a message tagging the specified "PVMer" role on Discord.
Automatic Data Population: Includes the player’s in-game name in the request.

This plugin streamlines the process of looking for PVM groups, making it easier for players to coordinate and find groups within their communities

@runelite-github-app
Copy link

runelite-github-app bot commented Feb 13, 2025

commas are the bane of my existence.
@Lorkrakt
Copy link
Author

This is ready for review whenever someone's ready.

cleaned up licensing on files. dont need on individual files. was redundant.
@LlemonDuck
Copy link
Contributor

Required:

  • Use an injected OkHttpClient to asynchronously send the discord payload instead of performing the i/o on swing thread

Recommended:

  • Use ImageUtil#loadImageResource and ImageUtil#resizeImage to simplify your nav button image loading, optionally just not resizing at all since the icon is already 16x16
  • Use injector.getInstance(lfghelperpanel.class); instead of manually instantiating the panel, which allows you to @Inject fields into it
  • Use gson to build your discord payload which will prevent text-escaping issues

@LlemonDuck LlemonDuck added the waiting for author waiting for the pr author to make changes or respond to questions label Feb 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
plugin added waiting for author waiting for the pr author to make changes or respond to questions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants