Skip to content

阈值对齐后 hot_leads 与 high_priority 的过滤条件逐字相同:两个视图是否还该并存是产品决策 #775

Closed

Description

@yinlianghui

来源:#766 的落地(PR #774)。观察类,不是用户今天会撞上的缺陷,因此打 finding、不入 pm:queue,留给维护者定夺。

现状

#766 把 hot_leads 的评分阈值从 >= 4.5 对齐到路由流的 >= 4(一个 hot 定义)。代价是它与 high_priority 的过滤条件此后逐字相同:

视图 filter(PR #774 之后)
hot_leads(🔥 Hot Leads) rating >= 4 + status in (new, contacted)
high_priority(High Priority) rating >= 4 + status in (new, contacted)

差异只剩表现层:

  • hot_leads:列含 phone / email / next_followup_date / owner_id,按 next_followup_date 升序(路由刚盖的 SLA),是工作队列;
  • high_priority:列含 lead_source,rowColor 按评分着色(五星绿 / 四星橙),无排序声明,是速览表。

线索列表没有 tabs,切换器把全部 8 个视图都列出来,所以一条四星线索现在同时出现在这两个页签里,两张表的行集合完全一致。

为什么单独立单,而不是在 #766 里顺手处理

#766 的分诊裁定范围是「hot 只有一个定义,视图跟随流」,并明确指出五星专属队列若是真实需求应另立视图。至于并存本身是否还合理,是产品决策,#766 的正文也把它列为 A 方向的遗留问题(「需要重新回答这两个视图是否还该并存」)。PR #774 因此只做对齐,没有删改 high_priority,并把两者的用途差异写进了视图注释与 content/docs/sales/leads.mdx(三语言)。

可选方向(都需要产品决定,勿直接猜)

  1. 保持现状:承认两者是同一批人的两种看法(工作序 vs 速览),文档已按此口径写。成本是切换器里两个页签给出同样的行。
  2. 合并:删掉其中一个,把另一个的列 / 着色并进来。要回答的是「谁被删」以及四个语言包 _views 里对应 label 的处理(另见 en.ts 的 _views 少了三个视图标签(renewals_due / at_risk_accounts / hot_leads),而其余三个语言包都有;_views 没有任何跨语言覆盖守卫 #767:en.ts 的 _views 本来就缺 hot_leads 标签)。
  3. 重新差异化 high_priority:给它一个真正不同的判据(例如不限状态、或按 annual_revenue / 行业切),使它不再是 hot 的同集。注意:任何回到「用不同评分阈值区分」的做法都会重新引入 🔥 Hot Leads 视图的 rating ≥ 4.5 与整星评分/路由流的「hot = rating ≥ 4」不一致:被判定为 hot、发了告警的四星线索永远进不了这个队列 #766 的缺陷。

若最终选 2 或 3,test/hot-lead-threshold-parity.test.ts 的钉子(视图阈值 == 流 hot 分支阈值)仍然有效,只要 hot_leads 还在。

Refs #766, #774

Activity

  1. added
    metadataDeclarative metadata — schema, security posture, UI surfaces
    on Aug 5, 2026
  2. added
    needs-user-decisionNeeds the maintainer's call before work proceeds
    and removed
    metadataDeclarative metadata — schema, security posture, UI surfaces
    on Aug 5, 2026
  3. yinlianghui commented on Aug 5, 2026

    @yinlianghui
    ContributorAuthor

    📋 findings 分诊轮(修复线 PM):晋级 needs-user-decision——hot_leads 与 high_priority 完全重叠后是否并存(保留/合并/按非评分维度重差异化)是产品决策,归收件箱。#787 落地的守卫已防止两者再悄悄分叉,不急。


    Generated by Claude Code

  4. huangyiirene commented on Aug 11, 2026

    @huangyiirene
    Collaborator

    Ruling (maintainer, 2026-08-11, PM chat, verbatim: 「接受你的全部建议」): Option 1 — both views stay. The row sets are identical but the presentations serve different jobs (hot_leads = work queue with follow-up ordering; high_priority = glance view with rating colors), and the docs already describe them on that口径. Merging or re-differentiating is the costly move; keeping them costs nothing. test/hot-lead-threshold-parity.test.ts stays as the guard against the thresholds drifting apart again. Closing as decided.


    Generated by Claude Code

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions