Skip to content

DEV: Remove the legacy widget code#135

Merged
megothss merged 7 commits intomainfrom
dev/purge-widgets
Nov 26, 2025
Merged

DEV: Remove the legacy widget code#135
megothss merged 7 commits intomainfrom
dev/purge-widgets

Conversation

@megothss
Copy link
Contributor

Remove deprecated discourse.post-stream-widget-overrides and associated legacy widget customizations.

This cleanup reduces technical debt, ensures future maintainability, and aligns with the latest framework improvements.

Remove deprecated `discourse.post-stream-widget-overrides` and associated
legacy widget customizations.

This cleanup reduces technical debt, ensures future maintainability, and
aligns with the latest framework improvements.
Remove the unused `PLUGIN_ID` constant in the `whos-online` API initializer
to reduce redundant code. This aligns with code cleanliness and maintainability
standards for the plugin.
Add compatibility entry for Discourse version `< 3.6.0.beta3-latest` in the
`discourse-whos-online` plugin. This ensures accurate version constraints
and prevents potential issues with unsupported Discourse versions.
@megothss megothss marked this pull request as ready for review November 21, 2025 03:08
Adjust the `.discourse-compatibility` file to reflect compatibility with
`< 2025.12.0-latest`, ensuring the plugin aligns with upcoming core
versioning changes. This update preempts potential mismatches and maintains
plugin functionality with evolving Discourse core updates.
Eliminate the unused `appEvents` service injection in `WhosOnlineService`
to clean up the code and improve maintainability. This reduces unnecessary
dependencies and aligns with recent refactoring efforts in the plugin.
@megothss megothss merged commit ab2d608 into main Nov 26, 2025
5 checks passed
@megothss megothss deleted the dev/purge-widgets branch November 26, 2025 18:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants