mailpace / mailpace-rails

A plugin for ActionMailer to send emails via MailPace.com
https://mailpace.com
MIT License
40 stars 2 forks source link

Gem should declare dependency on actionmailbox or set it as optional #27

Closed Fire-Dragon-DoL closed 1 month ago

Fire-Dragon-DoL commented 11 months ago

Hi! Just wanted to let you know that the gem should declare a dependency on actionmailbox, my rails installation doesn't use actionmailbox and activestorage and as such, are disabled. In development, this goes unnoticed (due to lazy load), however in production this caused a small outage because activestorage couldn't be loaded

paul-oms commented 1 month ago

Thanks for the feedback and apologies for the delay in getting back (notifications on this repo were disabled).

Apologies for causing an outage, I know how annoying that can be. This is now fixed in https://github.com/mailpace/mailpace-rails/commit/a8a15d0156ab9c70dadec56b5bf846853e4018a2 and will be in the next version