Skip to content

Conversation

coolsam726
Copy link
Owner

Improvements and Bug Fixes: Definition of Closures

  • Moved closures for getting the user name, avatar and mention items to the commentable model
  • Override getUserAvatar and getUserName in the commentable model to customize these.
  • Override getMentionsQuery to modify the query used to get mentions in the commentable model
  • Removed closures from the config to allow optimization through cache.
  • Bug Fix: Mentions item name was using the currently logged in user
  • New Component: Comment Card view component
  • New Component: Comment Card Entry

coolsam726 and others added 4 commits June 5, 2025 06:18
- Evaluate closures at the card level
- Added a Panel that uses Repeatable Entry
- TODO: Fix the comments avatars and user names
- Moved closures for getting the user name, avatar and mention items to the commentable model
- Override getUserAvatar and getUserName in the commentable model to customize these.
- Override getMentionsQuery to modify the query used to get mentions in the commentable model
- Removed closures from the config to allow optimization through cache.
- Bug Fix: Mentions item name was using the currently logged in user
- New Component: Comment Card view component
- New Component: Comment Card Entry
@coolsam726 coolsam726 enabled auto-merge June 6, 2025 22:35
@coolsam726 coolsam726 merged commit f242675 into main Jun 6, 2025
27 checks passed
@coolsam726 coolsam726 deleted the features branch June 6, 2025 22:37
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