[Bug]: Contextual bar actions do not work and duplicate reply names are allowed during edits
还没有人认领这个 Issue。
- #71 来自 @sezallagwal —— 已关闭，未合并
评估
- 难度
- 3/5
- 预计耗时
- 1-2 天
- 新手友好度
- 48/100
- Issue 类型
- 缺陷
- 描述清晰度
- 描述清楚
- 活跃度
- 停滞
- 技术栈
- typescript
- 领域
- full-stack
调研方向
从 ExecuteBlockActionHandler.ts 开始，跟踪上下文栏交互，并在 ReplyStorage.ts 中检查编辑期间的 isUniqueReplyName。重现 /quick list 中的 Send、Edit 和 Delete 操作，然后尝试将回复重命名为已有的名称。所有操作都能执行，并且编辑期间会拒绝重复名称，即表示完成。
由索引模型根据 Issue 内容生成。
描述
What happened?
Overview
This issue covers two distinct bugs within the quick replies list (/quick list) and the editing flow.
Bug 1: Send, Edit, and Delete actions in quick replies list do nothing
When opening the quick replies list and clicking Send, Edit, or Delete from the overflow menu (⋮) on any reply, nothing happens — no modal opens, no error is shown.
Root Cause:
In ExecuteBlockActionHandler.ts, the room fallback uses strict equality:
if (room === undefined) {
For contextual bar block interactions, getInteractionData().room returns null (not undefined). The strict check misses null, forcing room to stay null. Following this, the if (room) guard around the Send/Edit/Delete switch evaluates to false, silently skipping the execution of all three actions.
Bug 2: Duplicate reply names allowed when editing
When editing a reply and changing its name to perfectly match another existing reply, the app accepts it and saves it instead of showing a validation error.
Root Cause:
In ReplyStorage.ts, the isUniqueReplyName method has a no-op comparison:
if (replyFound && replyId) {
replyFound.id === replyId; // comparison result is discarded!
return true; // always returns true
}
Steps to reproduce
- Open any channel and run
/quick list. - Click the overflow menu (⋮) on any reply.
- Click Send, Edit, or Delete.
- Expected: A modal opens for the selected action.
- Actual: Nothing happens.
Relevant ScreenShots
No response
Version
8.1.1
What browsers are you seeing the problem on?
Chrome
- 主要语言
- TypeScript
- 星标
- 7
- 派生
- 21
- PR 合并指标
- 30 天内没有已合并 PR
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 有 Pull Request 模板
- 没有贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
RocketChat/Apps.QuickReplies 的其他 Issue
-
[Bug] Handle Gemini error responses safely in /quick ai可能已有人在做 @CodeMatrix1 于 230 天前认领。 未关闭bug
难度 3/5 1-2 天 新手友好度 52/100
RocketChat/Apps.QuickReplies#82 · 1 条评论 ·
-
[Bug]: /quick ai error messages are not role-aware and lack actionable guidance可能重新可做 关联的 PR 已关闭且未合并。 未关闭bug
难度 4/5 3-5 天 新手友好度 45/100
-
bug
难度 3/5 1-2 天 新手友好度 45/100
-
[Feat] One Click AI-Powered Reply Suggestions for Improved UX可能重新可做 @moyrsd 于 583 天前认领，目前没有进行中的 PR。 未关闭enhancement
RocketChat/Apps.QuickReplies#59 · 3 条评论 · 已指派 1 人 ·
-
bug
难度 2/5 1-3 小时 新手友好度 35/100
RocketChat/Apps.QuickReplies#55 · 2 条评论 ·
查看 RocketChat/Apps.QuickReplies 的全部 Issue
相似的 Issue
-
Bug: Broken RTL / BiDi text rendering when mixing Persian (Farsi) with English or numbers可能已有人在做 关联的 PR 仍在进行中或已合并。 未关闭bug
难度 2/5 半天 新手友好度 68/100
amosblomqvist/learnharness#758 ·
维护者通常 2 天内回复
-
难度 2/5 1-3 小时 新手友好度 62/100
bitfocus/companion-module-renewedvision-propresenter-api#134 ·
-
难度 2/5 1-3 小时 新手友好度 82/100
rcpch/digital-growth-charts-react-component-library#304 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 72/100
dsh-market/dsh-market#1210 · 1 条评论 ·
维护者通常 1 天内回复
-
good first issue
难度 2/5 1-3 小时 新手友好度 86/100
trustjonathan/STUDY-HUB#13 ·