starred/cypht

Fork 0

mirror of https://github.com/cypht-org/cypht.git synced 2026-04-25 13:05:53 +03:00

[GH-ISSUE #465] Cypht doesn't detect my Drafts folder #373

New issue

Closed

opened 2026-02-25 21:34:52 +03:00 by kerem · 11 comments

kerem commented

2026-02-25 21:34:52 +03:00

Owner

Originally created by @neildarlow on GitHub (Apr 10, 2021).
Original GitHub issue: https://github.com/cypht-org/cypht/issues/465

Originally assigned to: @jasonmunro on GitHub.

🐛 Bugreport

RFC6154 describes a method for determining Special Folders assigned with Special Use flags.

I am using Dovecot and have set a folder Drafts with the Special Use flag of \Drafts. The Folder Settings page
doesn't display the ability to assign my Drafts folder although the code suggests it should.

Is it the case that the Drafts folder name is checked-for/set using a hard-coded value? I see it referred to in code as 'draft'.

Version & Environment

Rev: [1accf48] - master branch

OS: [archlinuxARM/PHP-8.0.3]

Steps to reproduce

Navigate to Folder Settings page.
Select IMAP server
Note that the page doesn't display a means to assign the Drafts folder which does exist.

Originally created by @neildarlow on GitHub (Apr 10, 2021). Original GitHub issue: https://github.com/cypht-org/cypht/issues/465 Originally assigned to: @jasonmunro on GitHub. ## 🐛 Bugreport  RFC6154 describes a method for determining Special Folders assigned with Special Use flags. I am using Dovecot and have set a folder **Drafts** with the Special Use flag of \Drafts. The Folder Settings page doesn't display the ability to assign my Drafts folder although the code suggests it should. Is it the case that the Drafts folder name is checked-for/set using a hard-coded value? I see it referred to in code as **'draft'**. ### Version & Environment  Rev: [1accf48] - master branch  OS: [archlinuxARM/PHP-8.0.3] ### Steps to reproduce 1. Navigate to Folder Settings page. 2. Select IMAP server 3. Note that the page doesn't display a means to assign the **Drafts** folder which does exist.

kerem

2026-02-25 21:34:52 +03:00

closed this issue
added the
bug

imap_folders
labels

kerem commented

2026-02-25 21:34:53 +03:00

Author

Owner

@henrique-borba commented on GitHub (Apr 12, 2021):

@neildarlow I'm solving the IMAP Draft support in https://github.com/jasonmunro/cypht/pull/464

It won't take long 🙂

@henrique-borba commented on GitHub (Apr 12, 2021): @neildarlow I'm solving the IMAP Draft support in https://github.com/jasonmunro/cypht/pull/464 It won't take long 🙂

kerem commented

2026-02-25 21:34:53 +03:00

Author

Owner

@neildarlow commented on GitHub (Apr 12, 2021):

@henrique-borba Thank you for this. I hope I can find out what's going on with JavaScript on my PHP-8.0.3 setup so I can test your fix.

@neildarlow commented on GitHub (Apr 12, 2021): @henrique-borba Thank you for this. I hope I can find out what's going on with JavaScript on my PHP-8.0.3 setup so I can test your fix.

kerem commented

2026-02-25 21:34:53 +03:00

Author

Owner

@henrique-borba commented on GitHub (Apr 12, 2021):

@henrique-borba Thank you for this. I hope I can find out what's going on with JavaScript on my PHP-8.0.3 setup so I can test your fix.

What's the problem with the Javascript? Please also tell me your browser and version.

You are more than welcome 😃

@henrique-borba commented on GitHub (Apr 12, 2021): > @henrique-borba Thank you for this. I hope I can find out what's going on with JavaScript on my PHP-8.0.3 setup so I can test your fix. What's the problem with the Javascript? Please also tell me your browser and version. You are more than welcome 😃

kerem commented

2026-02-25 21:34:53 +03:00

Author

Owner

@marclaporte commented on GitHub (Apr 13, 2021):

yay, we found one more tester for the upcoming draft feature :-)

@marclaporte commented on GitHub (Apr 13, 2021): yay, we found one more tester for the upcoming draft feature :-)

kerem commented

2026-02-25 21:34:53 +03:00

Author

Owner

@neildarlow commented on GitHub (Apr 13, 2021):

Tested. Partially working?

You can select the server and assign folders but they don't persist over logout and login. Nottifcation shows folder assigned.

That is, however, some progress.

@neildarlow commented on GitHub (Apr 13, 2021): Tested. Partially working? You can select the server and assign folders but they don't persist over logout and login. Nottifcation shows folder assigned. That is, however, some progress.

kerem commented

2026-02-25 21:34:53 +03:00

Author

Owner

@jasonmunro commented on GitHub (Apr 13, 2021):

@neildarlow nothing persists between logins unless you explicitly save your settings first. I just tested to make sure and it worked fine as long as I go to Settings -> Save.

@jasonmunro commented on GitHub (Apr 13, 2021): @neildarlow nothing persists between logins unless you explicitly save your settings first. I just tested to make sure and it worked fine as long as I go to Settings -> Save.

kerem commented

2026-02-25 21:34:53 +03:00

Author

Owner

@neildarlow commented on GitHub (Apr 13, 2021):

Doh, of course! Logout didn't prompt me to save though.

That was a really simple fix. ;-)

@neildarlow commented on GitHub (Apr 13, 2021): Doh, of course! Logout didn't prompt me to save though. That was a really simple fix. ;-)

kerem commented

2026-02-25 21:34:53 +03:00

Author

Owner

@jasonmunro commented on GitHub (Apr 13, 2021):

Good point it did not. Because of how that page works we need 1 full refresh to enable the prompt on logout. I will see if that can be improved :). Thanks for the report and feedback!

@jasonmunro commented on GitHub (Apr 13, 2021): Good point it did not. Because of how that page works we need 1 full refresh to enable the prompt on logout. I will see if that can be improved :). Thanks for the report and feedback!

kerem commented

2026-02-25 21:34:53 +03:00

Author

Owner

@marclaporte commented on GitHub (Apr 20, 2021):

@neildarlow Please test https://github.com/jasonmunro/cypht/pull/464

@marclaporte commented on GitHub (Apr 20, 2021): @neildarlow Please test https://github.com/jasonmunro/cypht/pull/464

kerem commented

2026-02-25 21:34:53 +03:00

Author

Owner

@henrique-borba commented on GitHub (May 10, 2021):

I think https://github.com/jasonmunro/cypht/pull/464 and https://github.com/jasonmunro/cypht/pull/474 solved this issue

@henrique-borba commented on GitHub (May 10, 2021): I think https://github.com/jasonmunro/cypht/pull/464 and https://github.com/jasonmunro/cypht/pull/474 solved this issue

kerem commented

2026-02-25 21:34:53 +03:00

Author

Owner

@neildarlow commented on GitHub (Jun 12, 2021):

I think we can close this issue as fixed. Special folders handling has been addressed.

@neildarlow commented on GitHub (Jun 12, 2021): I think we can close this issue as fixed. Special folders handling has been addressed.

kerem referenced this issue

2026-02-25 21:36:15 +03:00

[PR #373] [MERGED] [Privacy] Don't send referrer information when clicking links #795