-
Notifications
You must be signed in to change notification settings - Fork 11
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
#3347 no send empty list #3455
#3347 no send empty list #3455
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.
delay(priority: 10). | ||
seller_order_updated(order, supplier) | ||
else | ||
Rollbar.warning("Warning: Trying to send update email (for order id ##{order.id}) to supplier (#{supplier.id} - #{supplier.name}) with 0 users") |
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.
@ivar I think this warning should be removed. In some markets, there are organizations that are managed by the market manager. For example, you have a farm who can't handle technology so they just call a market manager on the phone and he/she does the order management for them. In this case, an organization could have an empty users list.
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.
@rbarreca - updated as per your feedback. Please review
@rbarreca extra space removed |
See issue at #3347
seller_order_update_no_recipient_3347
branchThe following specs have been reviewed and run green
spec/interactors/send_update_emails_spec.rb
spec/mailers/order_mailer_spec.rb
spec/models/order_spec.rb