Skip to content

Additional Webhook Features#77

Open
NotReeceHarris wants to merge 3 commits into
cbrewitt:mainfrom
NotReeceHarris:main
Open

Additional Webhook Features#77
NotReeceHarris wants to merge 3 commits into
cbrewitt:mainfrom
NotReeceHarris:main

Conversation

@NotReeceHarris

Copy link
Copy Markdown

While using this plugin, I noticed that when I lose focus on the client (for example, when I’m away from my PC), I sometimes miss important buy/sell timings, which can negatively impact hourly profit.

This pull request introduces additional webhook notification features to help address that issue.

image

Changes

Expanded Webhook Notifications

Added new webhook events to provide more granular alerts, including:

  • Buy
  • Sell
  • Dump
  • Abort/Collect

Existing stat updates (unchanged)

These notifications aim to ensure important actions are not missed, particularly when multitasking or away from the client.

Focus-Based Notifications

To reduce notification spam, I’ve added an “Only alert when unfocused” option.

  • When enabled, notifications are sent only if the client window is not focused.
  • This is implemented using an AWT focus event listener to determine the window’s focus state.

This helps prevent excessive alerts while actively playing, while still ensuring important events are captured when unfocused.

Mention Support

Added an optional mention text field for webhook messages.
This allows users to include a mention (e.g. @username or role mention) in the webhook payload for increased visibility.

image image image image

This is my 3rd time using JAVA so the code may need refactoring a little, i tried to use production ready practises but i may have missed some

@NotReeceHarris

NotReeceHarris commented Feb 17, 2026

Copy link
Copy Markdown
Author

Looked at the way the current tray notification system works its probably better to tie in with the current tray notification system then using our own AWT event listener

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