Clean up CodeCorps.Emails.Transmissions.Receipt
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 45/100
Piste de recherche
Commencez par lire CodeCorps.Emails.Transmissions.Receipt.build et le gestionnaire d’événements qui envoie l’e-mail, puis comparez leur gestion des erreurs avec celle d’autres implémentations d’e-mail. C’est terminé lorsque Receipt est infaillible, que le gestionnaire d’événements prend en charge les points de défaillance pertinents et que le comportement défensif a été résolu délibérément plutôt que dupliqué.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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.
- Langage dominant
- Elixir
- Étoiles
- 234
- Forks
- 82
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de code-corps/code-corps-api
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 15/100
code-corps/code-corps-api#1385 ·
-
Cannot Create new GitHub AppOuverte
Difficulté 3/5 1-2 jours Accessibilité débutants 25/100
code-corps/code-corps-api#1384 ·
-
Track anon Users that registerOuverte
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
code-corps/code-corps-api#1383 · 2 commentaires ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 35/100
code-corps/code-corps-api#1382 · 2 commentaires ·
-
Difficulty: Medium Skill: Elixir Time: 4 hours
code-corps/code-corps-api#1372 · 3 commentaires · 1 personne assignée ·
Toutes les issues de code-corps/code-corps-api
Issues similaires
-
test(core_elx): flaky remote-desktop provider teardown test asserts before teardown completesOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
carverauto/serviceradar#5128 ·
Les mainteneurs répondent en général sous 1 jour
-
V2 JSON serializer crashes channel and CDC processesPeut-être pris @mandar1045 l’a pris il y a 1 jour. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
Les mainteneurs répondent en général sous 1 jour
-
Handle short ciphertext in AES-GCM Decrypt instead of panickingPeut-être pris @pamod-madubashana l’a pris il y a 2 jours. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 92/100
semaphoreio/semaphore#1305 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
QuinnWilton/argus#5 · 1 commentaire ·
-
help-wanted L: docker L: elm L: github:actions L: helm L: ruby:bundler
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
dependabot/dependabot-core#16425 ·
Les mainteneurs répondent en général sous 2 jours