Skip to content

DEV: Add compatibility with the Glimmer Post Stream #127

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 6 commits into from
Jun 25, 2025

Conversation

megothss
Copy link
Contributor

@megothss megothss commented Jun 13, 2025

This PR updates the discourse-user-notes plugin to ensure compatibility with the new Glimmer-based Post Stream in Discourse. The changes include:

  • Refactoring user note components and interactions to work seamlessly with Glimmer rendering.
  • Resolving any issues arising from differences between legacy and Glimmer Post Stream APIs or lifecycle hooks.
  • Ensuring user notes display and functionality remain intact when the Glimmer Post Stream is active.

megothss added 4 commits June 13, 2025 19:04
Refactored the user notes plugin by splitting logic into multiple files, introducing value transformers, and adopting Glimmer components. Added a new setting `user_notes_icon_placement` to control icon placement (options: `name` or `avatar`).

[skip ci]
- Added functionality to display user notes count based on `user_custom_fields`.
- Updated `PostMetadataUserNotes` component to dynamically toggle between emoji or icon based on site settings.
- Refactored logic for rendering the user notes icon in different UI contexts (mobile and desktop).
- Removed hard-coded version constraint for `withPluginApi`
Extracted the logic for updating the `user_notes_count` field into a new `updatePostUserNotesCount` function. Updated all related components and initializers to use this function, reducing duplication and ensuring consistency in handling user notes updates.
@megothss megothss marked this pull request as ready for review June 25, 2025 02:43
Added a new compatibility entry for Discourse version 3.5.0.beta8-dev to ensure proper version tracking.
@megothss megothss merged commit a685386 into main Jun 25, 2025
5 checks passed
@megothss megothss deleted the dev/glimmer-post-stream branch June 25, 2025 12:32
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