notifier: Slack DM channel (depends on deeper Slack integration)
还没有人认领这个 Issue。
评估
- 难度
- 4/5
- 预计耗时
- 3-5 天
- 新手友好度
- 42/100
- Issue 类型
- 功能
- 描述清晰度
- 基本清楚
- 活跃度
- 冷清
- 技术栈
- typescript
调研方向
从 apps/api/src/notify/index.ts 和 issue #82 开始,了解现有的 notifier contract 和电子邮件流程。在选择 Slack credential 和 delivery 行为之前,阅读 specs/api/saml.md 和 specs/behaviors/help-wanted-roles.md。完成的标准是实现 Slack DM delivery、handle resolution、fallback 或 combined delivery、SLACK_BOT_TOKEN 配置,以及相应的 failure logging。
由索引模型根据 Issue 内容生成。
描述
Follow-up from #82
#82 wires up a real notifier for help-wanted-role events (interest expressed, role filled). The agreed-on first-cut shape is email-only via Resend — covered in that issue.
This issue tracks the Slack DM channel, deliberately deferred until we tackle deeper Slack integration.
Why deferred
The Notifier interface in apps/api/src/notify/index.ts already accepts maintainerSlackHandle per notification, so the data flow is ready. What's missing is the credential trust — sending a DM from a workspace bot requires either:
- A workspace-installed Slack app with
chat:writescope + a bot token (the easiest path). - SAML IdP-side identity assertions extended to support delegated Slack actions (much heavier).
We're already the SAML IdP for codeforphilly.slack.com (specs/api/saml.md), but that's an inbound SAML flow (user signs into Slack via us). Sending DMs out to Slack is the inverse — we'd be initiating a workspace action on a member's behalf. Cleaner to do that via a dedicated Slack app with its own token.
Scope (when picked up)
- Add a
SlackDmNotifierimplementation alongsideEmailNotifier. - A new env var (
SLACK_BOT_TOKEN) — sealed-secret in the cluster repo. - Map Slack handles → user IDs via
users.lookupByEmail(orusers.listcached) since handles can change. Skip if the handle doesn't resolve. - Compound notifier: try Slack DM first, fall back to email if Slack delivery fails or the recipient has no Slack handle. (Or run both — pick when implementing.)
- Failure logging at the same envelope as today's notifier.
Related
- #82 — email-only first cut
specs/behaviors/help-wanted-roles.md— what gets delivered
- 主要语言
- TypeScript
- 星标
- 1
- 派生
- 1
- 平均合并
- 1 天 20 小时
- 30 天内合并 PR
- 25
贡献指南
这个仓库没有索引到贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
CodeForPhilly/codeforphilly-ng 的其他 Issue
-
难度 2/5 1-3 小时 新手友好度 84/100
-
MarkdownEditor toolbar: use Radix Toolbar from radix-ui instead of the hand-rolled roving tabindex 未关闭enhancement
难度 2/5 1-3 小时 新手友好度 82/100
-
难度 2/5 1-3 小时 新手友好度 76/100
-
难度 2/5 1-3 小时 新手友好度 88/100
-
难度 2/5 1-3 小时 新手友好度 74/100
查看 CodeForPhilly/codeforphilly-ng 的全部 Issue
相似的 Issue
-
VerificationGate: ATTRIBUTION quote guard never matches a normal quotation (\b around the quote) 未关闭
难度 2/5 1-3 小时 新手友好度 75/100
danielmiessler/LifeOS#2234 ·
-
T: Bug
难度 2/5 1-3 小时 新手友好度 75/100
-
难度 2/5 1-3 小时 新手友好度 65/100
-
难度 1/5 1 小时以内 新手友好度 85/100
-
Mend: dependency security vulnerability untriaged
难度 2/5 1-3 小时 新手友好度 70/100