mirror of
https://github.com/cypht-org/cypht.git
synced 2026-04-25 13:05:53 +03:00
[GH-ISSUE #233] Combined Sent view not including sent folders exposed via SPECIAL-USE #194
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#194
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 @jasonmunro on GitHub (Nov 22, 2017).
Original GitHub issue: https://github.com/cypht-org/cypht/issues/233
Originally assigned to: @jasonmunro on GitHub.
The combined Sent page is not auto-discovering Sent folders for IMAP accounts that expose these folders via the SPECIAL-USE extension. This is because the source list built in imap_sent_source() is excluding accounts without a manually set Sent folder.
Note that if "auto-bcc" is enabled, the special folders ARE included because the INBOX for that account is included, which triggers the code that collects sent messages to check for the SPECIAL-USE folder. Should be pretty easy to fix (I think) in the Hm_Handler_imap_sent class in modules/imap/modules.php by always passing the "check for special use" folders when collecting sent messages.
@jasonmunro commented on GitHub (Nov 29, 2017):
Should be fixed in the master branch