-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Exclude content length errors from recipient_error prop #290
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a couple of questions on how this works.
@@ -144,4 +147,9 @@ def __eq__(self, other): | |||
|
|||
@property | |||
def recipient_error(self): | |||
# TODO: This is a bandaid solution. We need to establish why we are calling this Cell property on | |||
# Cells that do not represent a recipient value. | |||
if self.error is not None and "Some messages may be too long due to custom content." in self.error: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is this case not considered an error?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are there some tests that cover what should and should not be considered an error?
Summary | Résumé
The
recipient_error
property was falsely identifying certain row errors asrecipient_errors
and incorrectly marking rows as having recipient errors causing API to mis-interpret what error messages to return to the endpoint caller.Related Issues | Cartes liées
This PR addresses an issue that was blocking the release pipeline following bumps to Utils' version in API.
Test instructions | Instructions pour tester la modification
Release Instructions | Instructions pour le déploiement
None.
Reviewer checklist | Liste de vérification du réviseur