Clean up CodeCorps.Emails.Transmissions.Receipt

未关闭

#1,337 创建于 2017年12月22日 0 条评论 0 个 reaction 已指派 0 人在 GitHub 查看

还没有人认领这个 Issue。

评估

难度: 4/5
预计耗时: 3-5 天
新手友好度: 45/100

Issue 类型: 重构
描述清晰度: 需要澄清
活跃度: 停滞
技术栈: elixir
领域: backend

调研方向

先阅读 CodeCorps.Emails.Transmissions.Receipt.build 和发送电子邮件的事件处理程序，然后将它们的失败处理与其他电子邮件实现进行比较。完成的标准是 Receipt 不会失败，事件处理程序负责相关的失败点，并且防御性行为已经经过有意的处理，而不是重复实现。

由索引模型根据 Issue 内容生成。

描述

blocked needs clarification needs estimate

Problem

In #1332, and even before that, CodeCorps.Emails.Transmissions.Receipt.build is wrapped into a with statement with multiple points of failure.

The event handler code which sends this email also wrapps that call into multiple points of failure.

The border between these seems arbitrary and, more importantly, The behavior of the Receipt itself is inconsistent with all other emails.

We should move these points of failure into the event handler and keep the Receipt itself infallible.

Then, we should decide if the points of failure actually make sense or if it's too defensive. This is old code, so it's quite possible we made some mistakes there.

主要语言: Elixir
星标: 234
派生: 82
PR 合并指标: 30 天内没有已合并 PR

贡献指南

打开贡献指南

从这里开始

先读完整个 Issue，再读项目的贡献指南。
在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
Fork 仓库，在一个分支上完成修改。
提交 Pull Request，并在描述里引用这个 Issue 编号。

code-corps/code-corps-api 的其他 Issue

URGENT: Security vulnerability detected in infrastructure config 未关闭

难度 5/5 一周以上新手友好度 15/100

code-corps/code-corps-api#1385 · 2026年1月6日创建
Cannot Create new GitHub App 未关闭

难度 3/5 1-2 天新手友好度 25/100

code-corps/code-corps-api#1384 · 2019年12月12日创建
Track anon Users that register 未关闭

难度 4/5 3-5 天新手友好度 35/100

code-corps/code-corps-api#1383 · 2 条评论 · 2018年3月12日创建
Use Dependabot to automatically keep dependencies up-to-date 未关闭

难度 3/5 1-2 天新手友好度 35/100

code-corps/code-corps-api#1382 · 2 条评论 · 2018年3月11日创建
Write tests for GitHub.Sync.pull_request_event/1 未关闭

Difficulty: Medium Skill: Elixir Time: 4 hours

code-corps/code-corps-api#1372 · 3 条评论 · 已指派 1 人 · 2018年1月17日创建

查看 code-corps/code-corps-api 的全部 Issue

相似的 Issue

log-collector cert missing .svc.cluster.local SAN for cross-namespace OTLP access 未关闭

难度 2/5 1-3 小时新手友好度 75/100

carverauto/serviceradar#4596 · 2026年9月24日创建
extra_applications: [tools: :optional] crashes normalize_dep/1 (FunctionClauseError) 未关闭

难度 2/5 1-3 小时新手友好度 75/100

erlef/mix_sbom#157 · 1 条评论 · 2026年9月23日创建
Empty form title causes internal server error 未关闭

bug

难度 2/5 1-3 小时新手友好度 65/100

pentacent/keila#579 · 2026年9月23日创建
opencode ACP does not support session/set_model 未关闭

难度 2/5 1-3 小时新手友好度 75/100

agentjido/jido_harness#80 · 2026年9月23日创建
Add show_pagination configuration 未关闭

难度 2/5 1-3 小时新手友好度 70/100

sevenseacat/cinder#235 · 2026年9月23日创建

更多 Elixir Issue