Skip to content
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

Send emails asynchronously #417

Open
augnustin opened this issue Sep 9, 2019 · 4 comments
Open

Send emails asynchronously #417

augnustin opened this issue Sep 9, 2019 · 4 comments

Comments

@augnustin
Copy link

augnustin commented Sep 9, 2019

So, coherence is using swoosh to send emails.

I read from their docs, that there is no pre-defined behavior to send emails asynchronously, but to use elixir's Task.

But then, the method that is in charge of sending email in coherence is there and is not configurable.

So it means there is no way to send emails in the background but by forking the repo? Shouldn't there be an option for that? At least a small sendAsync: true option?

Has anyone had the issue and found a solution?

I am aware the repo is unmaintained, but I don't feel like cloning it: I am already maintaining my own fork of ex_admin, it is enough... 😓

Thanks

@Kurisu3
Copy link

Kurisu3 commented Oct 16, 2019

Maybe will you be insterested on moving to another library that is actively maintained?

@augnustin
Copy link
Author

I sure am! Will give it a try when I get some little time.

@nesimtunc
Copy link

I started to change every email sending function to make async using Task according to Swoosh's README.

But nothing changed, still sending slow and it doesn't send actually, even it logs sent. There must be a problem with my SMTP server.

Or since I'm a newbie in Elixir world, I might have failed with the implementation. Here you can see my changes.

nesimtunc@940f87b

@nesimtunc
Copy link

Maybe will you be insterested on moving to another library that is actively maintained?

It seems good one but I had problem after changed my db tables' default int primary key to uuid one.

pow-auth/pow#396

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants