Skip to content

DEV: Added compatibility with the Glimmer Post Stream #213

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

Merged
merged 5 commits into from
Jul 22, 2025

Conversation

megothss
Copy link
Contributor

@megothss megothss commented Jul 18, 2025

  • Refactored the docs-topic component to render the Post component directly instead of using MountWidget when displaying a topic post. This streamlines compatibility with Glimmer-based post streams.
  • Updated .discourse-compatibility to reflect support for Discourse core version 3.5.0.beta8-dev.
  • Adjusted test configuration in docs-user-status-test.js to explicitly enable user status.

megothss added 3 commits July 18, 2025 16:09
Updated the `docs-topic` component to leverage the `Post` component when `useGlimmerPostStream` is enabled.

Enhanced test configurations to include `enable_user_status` and adjusted factory setup in specs for consistency.
Added a directive to disable the `no-capital-arguments` linting rule for compatibility with `MountWidget`.
… 3.5.0.beta8-dev`

Added a new compatibility entry to the `.discourse-compatibility` file to ensure proper plugin functionality with the specified core version.
megothss added 2 commits July 21, 2025 22:24
Replaced conditional logic for `useGlimmerPostStream` with direct usage of the `Post` component, simplifying the `docs-topic` implementation.
Eliminated an unnecessary `MountWidget` import as it is no longer utilized in the `docs-topic` component.
@megothss megothss merged commit 2cfd050 into main Jul 22, 2025
5 checks passed
@megothss megothss deleted the dev/glimmer-post-stream branch July 22, 2025 02:10
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