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

Store email body on failed send #50

Closed
wants to merge 2 commits into from
Closed

Conversation

allmtz
Copy link
Contributor

@allmtz allmtz commented Jan 25, 2024

This is an implementation of #45

When an email fails to send, the body is saved to the /tmp/pop dir with a filename in the format y-m-d-timeStamp. I also updated the error message to display the path of the file created and increased the error display time.

It's now possible to run a command similar to what @maaslalani suggested.

pop < /tmp/pop/2024-1-24-61319

This prevents users having to retype lengthy emails when a send fails and pop must be closed to address the send error.

@maaslalani
Copy link
Contributor

This is awesome @allmtz, thank you so much for this great improvement!

@maaslalani
Copy link
Contributor

Hey @allmtz, thanks so much for this PR. I modified it a little bit in #64 and merged it! Thank you for all the work done on this one!

@maaslalani maaslalani closed this Apr 25, 2024
@allmtz
Copy link
Contributor Author

allmtz commented Apr 29, 2024

@maaslalani sure thing, glad I could help!

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

Successfully merging this pull request may close these issues.

2 participants