Skip to content

fix: hide reply button on archived channels#524

Merged
wesbillman merged 2 commits into
mainfrom
cant-reply
May 11, 2026
Merged

fix: hide reply button on archived channels#524
wesbillman merged 2 commits into
mainfrom
cant-reply

Conversation

@matt2e

@matt2e matt2e commented May 11, 2026

Copy link
Copy Markdown
Collaborator
Screenshot 2026-05-11 at 2 22 29 pm

Previously i would see in archived channels:

  • A bunch of actions on messages that I had send (react, edit, delete, reply). These would all fail with an error about the channel being archived
  • A reply button on messages from others

I've changed the logic so it hides these actions when the channel is archived

Pass `undefined` for `onReply` when the channel is archived, preventing
the Reply button from appearing in the message action bar. This mirrors
the existing pattern used for `onToggleReaction`.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Signed-off-by: Matt Toohey <contact@matttoohey.com>
@matt2e matt2e requested a review from wesbillman as a code owner May 11, 2026 04:15
Gate onDelete, onEdit, and onEditSave with the archivedAt check in
ChannelScreen, matching the existing effectiveToggleReaction pattern.
This ensures no action buttons appear on hover for archived channels.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Signed-off-by: Matt Toohey <contact@matttoohey.com>

@wesbillman wesbillman left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@wesbillman wesbillman merged commit ed9498e into main May 11, 2026
15 checks passed
@wesbillman wesbillman deleted the cant-reply branch May 11, 2026 14:56
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