Skip to content

Improve comment notifications to a content that was loaded already at somepoint#3430

Merged
tzarebczan merged 1 commit intomasterfrom
comment-notification-fix
Feb 25, 2026
Merged

Improve comment notifications to a content that was loaded already at somepoint#3430
tzarebczan merged 1 commit intomasterfrom
comment-notification-fix

Conversation

@keikari
Copy link
Contributor

@keikari keikari commented Feb 25, 2026

Multiple comment notifications to a content didn't worked well. Basically the comment notifications, after the first one, didn't load the linked comment anymore. This should fix.

Summary by CodeRabbit

Release Notes

  • New Features

    • Comments in linked chains now automatically expand replies when accessed.
  • Improvements

    • Optimized comment loading behavior for better performance.

@keikari keikari changed the title Improve comment notifications to a content that was loaded already at Improve comment notifications to a content that was loaded already at somepoint Feb 25, 2026
@coderabbitai
Copy link

coderabbitai bot commented Feb 25, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 6664ee2 and bd87e54.

📒 Files selected for processing (2)
  • ui/component/comment/view.jsx
  • ui/component/commentsList/view.jsx

📝 Walkthrough

Walkthrough

These changes enhance comment thread handling by automatically expanding replies in linked comment chains and refactoring the fetching logic for thread and linked comments into a dedicated effect with updated dependency arrays.

Changes

Cohort / File(s) Summary
Comment Chain Auto-Expansion
ui/component/comment/view.jsx
Adds a React effect that automatically expands replies (sets showReplies to true) and resets pagination to page 1 when a comment enters a linked comment chain.
Comments List Fetch Refactoring
ui/component/commentsList/view.jsx
Extracts thread and linked comment fetching into a dedicated effect with dependencies [fetchComment, linkedCommentId, threadCommentId], removing eager fetch logic from the main comments effect and updating cleanup dependencies.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 When linked chains dance in comment space,
A little effect springs into place—
Expanding replies with grace so true,
Dependencies cleaned, and flows made new! ✨

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch comment-notification-fix

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@tzarebczan tzarebczan merged commit 5bd4fee into master Feb 25, 2026
2 of 3 checks passed
@tzarebczan tzarebczan deleted the comment-notification-fix branch February 25, 2026 14:33
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.

2 participants