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

add inline (embedded) images support #8

Merged
merged 1 commit into from
Apr 24, 2022

Conversation

s0x90
Copy link
Contributor

@s0x90 s0x90 commented Apr 19, 2022

Support image embedding from attachments #3

@umputun
Copy link
Member

umputun commented Apr 20, 2022

you have not requested review for any of your current PRs, so I don't check them and treats as some kind of work in progress.

In case if you forgot to request and expects me to review - pls let me know

@s0x90
Copy link
Contributor Author

s0x90 commented Apr 20, 2022

Oh my mistake. @umputun pls check pr

@umputun
Copy link
Member

umputun commented Apr 21, 2022

I see nothing wrong with it. If you tested it end-to-end and it worked, I'll merge

@s0x90
Copy link
Contributor Author

s0x90 commented Apr 21, 2022

Everything seems correct. I have tested e2e with smtp.yandex.com and https://github.com/rnwood/smtp4dev

@s0x90
Copy link
Contributor Author

s0x90 commented Apr 23, 2022

@umputun do you expect me to do extra tests?

Copy link
Member

@umputun umputun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thx

@umputun umputun merged commit f81e515 into go-pkgz:master Apr 24, 2022
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