[PR #902] [MERGED] Fixed smtp compose form not working #1130

New issue

Closed

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

kerem commented

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

Owner

📋 Pull Request Information

Original PR: https://github.com/cypht-org/cypht/pull/902
Author: @josaphatim
Created: 2/16/2024
Status: ✅ Merged
Merged: 2/19/2024
Merged by: @Shadow243

Base: master ← Head: fix-smtp-composer

📝 Commits (2)

7e3e9f2 Fixed smtp compose form not working
7fd4041 Took in account attachment reminder

📊 Changes

3 files changed (+33 additions, -31 deletions)

View changed files

📝 modules/smtp/hm-smtp.php (+1 -1)
📝 modules/smtp/modules.php (+1 -1)
📝 modules/smtp/site.js (+31 -29)

📄 Description

Fixed sending mails with SMTP modules. Calling event.preventDefault() broke the form to be submitted to php. This MR adds a button to do checks and possibly launch the modal before the form is submitted.

_{🔄 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/902 **Author:** [@josaphatim](https://github.com/josaphatim) **Created:** 2/16/2024 **Status:** ✅ Merged **Merged:** 2/19/2024 **Merged by:** [@Shadow243](https://github.com/Shadow243) **Base:** `master` ← **Head:** `fix-smtp-composer` --- ### 📝 Commits (2) - [`7e3e9f2`](https://github.com/cypht-org/cypht/commit/7e3e9f25986de1b595d0bec42c8be44e5c05b1e4) Fixed smtp compose form not working - [`7fd4041`](https://github.com/cypht-org/cypht/commit/7fd404101965664484b545507c2ff54d54c5de43) Took in account attachment reminder ### 📊 Changes **3 files changed** (+33 additions, -31 deletions) <details> <summary>View changed files</summary> 📝 `modules/smtp/hm-smtp.php` (+1 -1) 📝 `modules/smtp/modules.php` (+1 -1) 📝 `modules/smtp/site.js` (+31 -29) </details> ### 📄 Description Fixed sending mails with SMTP modules. Calling event.preventDefault() broke the form to be submitted to php. This MR adds a button to do checks and possibly launch the modal before the form is submitted. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>