[PR #855] [MERGED] Refactor/stmp imap server configuration #1096

New issue

Closed

opened 2026-02-25 21:37:13 +03:00 by kerem · 0 comments

kerem commented

2026-02-25 21:37:13 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/cypht-org/cypht/pull/855
Author: @henochit
Created: 1/3/2024
Status: ✅ Merged
Merged: 3/22/2024
Merged by: @kroky

Base: master ← Head: refactor/stmp_imap_server_configuration

📝 Commits (10+)

6631757 [Ref]: Add stepper
82e20e3 Merge branch 'master' of https://github.com/jasonmunro/cypht into refactor/stmp_imap_server_configuration
1a5b863 [Ref]: Rename
2cfcd26 [feat] add validation on first step
f5c24fc [feat]: implement auto detect provider from email then load provider config details
5bef982 [feat]: set preconfigured value
4b25b6d [Feat]: add conditional stmp and imap display
ff65f40 [Feat]: Implement imap & smtp con.
443923b Feat: implement saving profile logic
02713a0 [Ref] implement profil saving & refactor nux by splitting code in functions

📊 Changes

26 files changed (+1613 additions, -602 deletions)

View changed files

📝 .gitignore (+1 -0)
📝 .travis/.env (+0 -3)
📝 language/en.php (+19 -1)
📝 modules/core/handler_modules.php (+146 -0)
📝 modules/core/output_modules.php (+189 -0)
📝 modules/core/setup.php (+42 -4)
📝 modules/core/site.css (+31 -0)
📝 modules/core/site.js (+302 -3)
📝 modules/imap/functions.php (+48 -0)
📝 modules/imap/handler_modules.php (+8 -8)
📝 modules/imap/output_modules.php (+133 -66)
📝 modules/imap/setup.php (+3 -4)
📝 modules/imap/site.css (+5 -5)
📝 modules/ldap_contacts/modules.php (+1 -1)
📝 modules/nux/modules.php (+27 -0)
📝 modules/nux/services.php (+6 -6)
📝 modules/nux/setup.php (+12 -2)
📝 modules/nux/site.js (+1 -1)
➕ modules/profiles/functions.php (+21 -0)
📝 modules/profiles/modules.php (+3 -2)

...and 6 more files

📄 Description

Goal: Make it easier to set up IMAP, SMTP and connect them together (A profile with optional signature)

Context: Cypht is very flexible but quite unintuitive/complicated to set up a simple email account. Too many steps. Too many places it can go wrong and you end up with a half configured system.

A related discussion to "use login data from IMAP for SMTP"
https://github.com/jasonmunro/cypht/issues/459

Also desired: auto-configuration via auto-detection or via an URL

Details (login required): reference

How2Test

Configure an SMTP and IMAP server at the same time (Enable Profile creation)
Configure an JMAP server
Configure an SMTP server only
Configure an IMAP server only

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/cypht-org/cypht/pull/855 **Author:** [@henochit](https://github.com/henochit) **Created:** 1/3/2024 **Status:** ✅ Merged **Merged:** 3/22/2024 **Merged by:** [@kroky](https://github.com/kroky) **Base:** `master` ← **Head:** `refactor/stmp_imap_server_configuration` --- ### 📝 Commits (10+) - [`6631757`](https://github.com/cypht-org/cypht/commit/66317578dcb37ab512c77c8e53ecb1796a26dce2) [Ref]: Add stepper - [`82e20e3`](https://github.com/cypht-org/cypht/commit/82e20e3a2bea90e869f12e9bf409af55e11bacf4) Merge branch 'master' of https://github.com/jasonmunro/cypht into refactor/stmp_imap_server_configuration - [`1a5b863`](https://github.com/cypht-org/cypht/commit/1a5b863ba97717a4ab5e7fa72b02dc3817ae0b1d) [Ref]: Rename - [`2cfcd26`](https://github.com/cypht-org/cypht/commit/2cfcd265acae4e9f73d434ffe4ac8d113931abd7) [feat] add validation on first step - [`f5c24fc`](https://github.com/cypht-org/cypht/commit/f5c24fc8c9fa06d3516515e6c7c4412769e4314d) [feat]: implement auto detect provider from email then load provider config details - [`5bef982`](https://github.com/cypht-org/cypht/commit/5bef982daa16bc3a38de5dee8b936b7dc554ea02) [feat]: set preconfigured value - [`4b25b6d`](https://github.com/cypht-org/cypht/commit/4b25b6d4047ca1a0ed49d384d319d3f1efc2ac21) [Feat]: add conditional stmp and imap display - [`ff65f40`](https://github.com/cypht-org/cypht/commit/ff65f40b44ee9a35e5d210906d57d2f58a66a97e) [Feat]: Implement imap & smtp con. - [`443923b`](https://github.com/cypht-org/cypht/commit/443923b28f88be691e55c71ad84425a23da7cc67) Feat: implement saving profile logic - [`02713a0`](https://github.com/cypht-org/cypht/commit/02713a0884a95b76cf16fcf0f9e1609056960190) [Ref] implement profil saving & refactor nux by splitting code in functions ### 📊 Changes **26 files changed** (+1613 additions, -602 deletions) <details> <summary>View changed files</summary> 📝 `.gitignore` (+1 -0) 📝 `.travis/.env` (+0 -3) 📝 `language/en.php` (+19 -1) 📝 `modules/core/handler_modules.php` (+146 -0) 📝 `modules/core/output_modules.php` (+189 -0) 📝 `modules/core/setup.php` (+42 -4) 📝 `modules/core/site.css` (+31 -0) 📝 `modules/core/site.js` (+302 -3) 📝 `modules/imap/functions.php` (+48 -0) 📝 `modules/imap/handler_modules.php` (+8 -8) 📝 `modules/imap/output_modules.php` (+133 -66) 📝 `modules/imap/setup.php` (+3 -4) 📝 `modules/imap/site.css` (+5 -5) 📝 `modules/ldap_contacts/modules.php` (+1 -1) 📝 `modules/nux/modules.php` (+27 -0) 📝 `modules/nux/services.php` (+6 -6) 📝 `modules/nux/setup.php` (+12 -2) 📝 `modules/nux/site.js` (+1 -1) ➕ `modules/profiles/functions.php` (+21 -0) 📝 `modules/profiles/modules.php` (+3 -2) _...and 6 more files_ </details> ### 📄 Description Goal: Make it easier to set up IMAP, SMTP and connect them together (A profile with optional signature) Context: Cypht is very flexible but quite unintuitive/complicated to set up a simple email account. Too many steps. Too many places it can go wrong and you end up with a half configured system. A related discussion to "use login data from IMAP for SMTP" https://github.com/jasonmunro/cypht/issues/459 Also desired: auto-configuration via auto-detection or via an URL Details (login required): [reference](https://avan.tech/item81120-Cypht-Make-it-easier-to-set-up-IMAP-SMTP-and-connect-them-together-A-profile-with-optional-signature) ### How2Test - Configure an SMTP and IMAP server at the same time (Enable Profile creation) - Configure an JMAP server - Configure an SMTP server only - Configure an IMAP server only --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>