mirror of
https://github.com/cypht-org/cypht.git
synced 2026-04-25 13:05:53 +03:00
[GH-ISSUE #1000] Being mistakenly warned about an empty email body #537
Labels
No labels
2fa
I18N
PGP
Security
Security
account
advanced_search
advanced_search
announcement
api_login
authentication
awaiting feedback
blocker
bug
bug
bug
calendar
config
contacts
core
core
devops
docker
docs
duplicate
dynamic_login
enhancement
epic
feature
feeds
framework
github
github
gmail_contacts
good first issue
help wanted
history
history
imap
imap_folders
inline_message
installation
keyboard_shortcuts
keyboard_shortcuts
ldap_contacts
mobile
need-ssh-access
new module set
nux
pop3
profiles
pull-request
question
refactor
release
research
saved_searches
smtp
strategic
tags
tests
themes
website
wordpress
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/cypht#537
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @PaulTGG on GitHub (May 3, 2024).
Original GitHub issue: https://github.com/cypht-org/cypht/issues/1000
Originally assigned to: @mercihabam on GitHub.
🐛 Bugreport
#866 was done to allow sending emails where there is no text in the body. Since then, the compose page has been updated, and now, it's possible to get the message warning of an empty email body when there actually is text in the body. Not sure if this also has something to do with the visual glitch where the word "message" appears over the toolbar (also seen in the screenshot below).
Version & Environment
Rev:
492f634Steps to reproduce
@mercihabam commented on GitHub (May 7, 2024):
PR https://github.com/cypht-org/cypht/pull/1006
@PaulTGG commented on GitHub (May 7, 2024):
I still have the same issue with the latest master as of this moment (

117f04b).@mercihabam commented on GitHub (May 7, 2024):
@PaulTGG Sure, the PR is still open. On master, you should still be getting the same behavior.
Maybe try it directly from the pull request?
@PaulTGG commented on GitHub (May 8, 2024):
@jacob-js Sorry about that - I wasn't paying close attention. I tried from the PR, and it works now, thanks!