Skip to content

Update Odablock Sounds Plugin#11888

Merged
riktenx merged 3 commits into
runelite:masterfrom
DapperMickie:odablock-update
May 17, 2026
Merged

Update Odablock Sounds Plugin#11888
riktenx merged 3 commits into
runelite:masterfrom
DapperMickie:odablock-update

Conversation

@DapperMickie
Copy link
Copy Markdown
Contributor

Fixes issue with Odablock Sounds Live Notifications, also adds 1 sounds for live notification.
Thanks @LogicalSoIutions

@runelite-github-app
Copy link
Copy Markdown

runelite-github-app Bot commented May 12, 2026

@LlemonDuck
Copy link
Copy Markdown
Contributor

add build=standard to runelite-plugin.properties

@LlemonDuck LlemonDuck added the waiting for author waiting for the pr author to make changes or respond to questions label May 12, 2026
@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 May 12, 2026
@DapperMickie
Copy link
Copy Markdown
Contributor Author

Done, now also includes a small config fix

@runelite-github-app
Copy link
Copy Markdown

runelite-github-app Bot commented May 12, 2026

Feedback:

  • javax.sound.sampled.UnsupportedAudioFileException: Use of javax.sound, use net.runelite.client.audio.AudioPlayer instead
  • javax.sound.sampled.LineUnavailableException: Use of javax.sound, use net.runelite.client.audio.AudioPlayer instead

A new scan will be automatically performed on the next push to the PR branch.


Internal use only: Reviewer details Maintainer details

@pajlada
Copy link
Copy Markdown
Contributor

pajlada commented May 13, 2026

DapperMickie/odablock-sounds@77e3b66914dc6c6725378887aa6f6b9e6898d226..DapperMickie:6ed6dc26efd4cee6d35632b0aaf226fa2a716f59#diff-c259c6cab6 this change is incompatible with how your plugin was previously configured.
You need to revert this back to a raw.githubusercontent.com url, or make live stream checking opt-in with a warning annotation.

@pajlada pajlada added the waiting for author waiting for the pr author to make changes or respond to questions label May 13, 2026
@LogicalSoIutions
Copy link
Copy Markdown
Contributor

What would the opt-in warning have to say? There is no data being sent to the servers, only incoming so I assumed I didn't need a warning, shame on me for assuming. (I was the one who made the changes hence why I am asking)

@coopermor
Copy link
Copy Markdown
Contributor

warning = "This feature will submit your IP address to a 3rd-party server not controlled or verified by Runelite developers."

@pajlada
Copy link
Copy Markdown
Contributor

pajlada commented May 13, 2026

Yep, what coopermor shared above is correct.
Keep in mind that adding that alone would not make the feature opt-in - users who have previously installed the plugin would still have the live stream feature installed & sending requests to a non-verified host without any recourse. You would need to reset the user's live-stream setting (i.e. by changing the key name) and changing the default value to false.

@LogicalSoIutions
Copy link
Copy Markdown
Contributor

Just going to go the GH route. Got the URL updated, just have to wait on Dapper to merge PR and update.

Thank you!

@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 May 13, 2026
@DapperMickie
Copy link
Copy Markdown
Contributor Author

Has been updated and reverted back to gh url

@riktenx riktenx added bug Something isn't working and removed bug Something isn't working labels May 17, 2026
@riktenx riktenx merged commit 9bf1335 into runelite:master May 17, 2026
2 of 3 checks passed
@DapperMickie DapperMickie deleted the odablock-update branch May 17, 2026 17:20
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.

6 participants