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

Empty message body results in two errors #1052

Closed
Zero3K opened this issue Mar 23, 2023 · 9 comments
Closed

Empty message body results in two errors #1052

Zero3K opened this issue Mar 23, 2023 · 9 comments

Comments

@Zero3K
Copy link

Zero3K commented Mar 23, 2023

The errors are "Can't save message: Message body is empty" (which happens when it autosaves the message and when clicking the Save button) and "Message body is empty" when trying to send the message.

@the-djmaze
Copy link
Owner

Correct, this is due to #974
SnappyMail now throws error when empty, else encrypt/sign will fail.

@Zero3K
Copy link
Author

Zero3K commented Mar 28, 2023

I do not have either option enabled.

the-djmaze pushed a commit that referenced this issue Apr 4, 2023
@Zero3K
Copy link
Author

Zero3K commented Apr 4, 2023

What about the adding the ability to send an email with no message but with one or more attachments?

@the-djmaze
Copy link
Owner

Could make it a warning with question yes/no

@Zero3K
Copy link
Author

Zero3K commented Apr 4, 2023

What is wrong with having no warning if someone has added attachments to the email?

@the-djmaze
Copy link
Owner

What is wrong with having no warning if someone has added attachments to the email?

None, but is it always intended?
Maybe it is by accident?

@chodid
Copy link

chodid commented Apr 4, 2023

Could make it a warning with question yes/no

Just stumbled upon this because I sometimes do want to send mails with attachments but no text. In general I would opt for the question option too - but it'd be great if this question could then be confirmed by simply hitting enter 😊

@Zero3K
Copy link
Author

Zero3K commented Jun 5, 2023

Can this issue be closed as complete now?

@the-djmaze
Copy link
Owner

Yes 😉

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