Clean up CodeCorps.Emails.Transmissions.Receipt

Abierto

#1,337 abierto el 22/12/2017 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad: 4/5
Tiempo estimado: 3-5 días
Aptitud para principiantes: 45/100

Tipo de issue: Refactorización
Claridad: Necesita aclaración
Estado de actividad: Estancado
Stack tecnológico: elixir
Área: backend

Línea de trabajo

Empieza leyendo CodeCorps.Emails.Transmissions.Receipt.build y el manejador de eventos que envía el correo electrónico; después, compara su gestión de errores con la de otras implementaciones de correo electrónico. Se considera terminado cuando Receipt es infalible, el manejador de eventos se hace cargo de los puntos de fallo relevantes y el comportamiento defensivo se ha resuelto deliberadamente en lugar de duplicarse.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

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.

Lenguaje dominante: Elixir
Estrellas: 234
Forks: 82
Métricas de merge de PR: Sin PR fusionados en 30 d

Guía de contribución

Abrir la guía de contribución

Primeros pasos

Lee el issue completo y luego la guía de contribución del proyecto.
Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
Haz un fork del repositorio y trabaja en una rama.
Abre un pull request que haga referencia al número del issue.

Más de code-corps/code-corps-api

URGENT: Security vulnerability detected in infrastructure config Abierto

Dificultad 5/5 Más de una semana Aptitud para principiantes 15/100

code-corps/code-corps-api#1385 · Abierto el 6/1/2026
Cannot Create new GitHub App Abierto

Dificultad 3/5 1-2 días Aptitud para principiantes 25/100

code-corps/code-corps-api#1384 · Abierto el 12/12/2019
Track anon Users that register Abierto

Dificultad 4/5 3-5 días Aptitud para principiantes 35/100

code-corps/code-corps-api#1383 · 2 comentarios · Abierto el 12/3/2018
Use Dependabot to automatically keep dependencies up-to-date Abierto

Dificultad 3/5 1-2 días Aptitud para principiantes 35/100

code-corps/code-corps-api#1382 · 2 comentarios · Abierto el 11/3/2018
Write tests for GitHub.Sync.pull_request_event/1 Abierto

Difficulty: Medium Skill: Elixir Time: 4 hours

code-corps/code-corps-api#1372 · 3 comentarios · 1 asignado · Abierto el 17/1/2018

Todos los issues de code-corps/code-corps-api

Issues similares

log-collector cert missing .svc.cluster.local SAN for cross-namespace OTLP access Abierto

Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100

carverauto/serviceradar#4596 · Abierto el 24/9/2026
extra_applications: [tools: :optional] crashes normalize_dep/1 (FunctionClauseError) Abierto

Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100

erlef/mix_sbom#157 · 1 comentario · Abierto el 23/9/2026
Empty form title causes internal server error Abierto

bug

Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100

pentacent/keila#579 · Abierto el 23/9/2026
opencode ACP does not support session/set_model Abierto

Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100

agentjido/jido_harness#80 · Abierto el 23/9/2026
Add show_pagination configuration Abierto

Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100

sevenseacat/cinder#235 · Abierto el 23/9/2026

Más issues de Elixir