-
Notifications
You must be signed in to change notification settings - Fork 342
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
SentEmailViewer should work with with all adapters. #518
Comments
Thanks for mentioning this @seivan . I'm not a 100% sure, but I think you could make this work already as is by using Bamboo.SentEmail? That's what the So if you wrote your own adapter that uses |
@germsvel Sorry for the delay. So I am wondering if it's possible to add the option to enable |
It would be nice if
SentEmailViewer
still worked with other adapters, maybeLocalAdapter
could be an add-on and not just the only one used.The rational is that we're using SES Mailbox Simulator for non prod environments, and would love to be able to use it while still having access to the
SentEmailViewer
.My suggestion is that
LocalAdapter
should not be a replacement for an actual adapter, but an addition.Is this something that's acceptable? In which case I don't mind setting up a PR for it.
The text was updated successfully, but these errors were encountered: