mirror of
https://github.com/cypht-org/cypht.git
synced 2026-04-25 13:05:53 +03:00
[PR #1311] [MERGED] Removed forget/save actions #1424
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#1424
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?
📋 Pull Request Information
Original PR: https://github.com/cypht-org/cypht/pull/1311
Author: @josaphatim
Created: 11/1/2024
Status: ✅ Merged
Merged: 11/4/2024
Merged by: @kroky
Base:
master← Head:removed-save-forgot-server-buttons📝 Commits (1)
b34489dRemoved forget/save actions📊 Changes
8 files changed (+28 additions, -298 deletions)
View changed files
📝
modules/imap/handler_modules.php(+10 -114)📝
modules/imap/output_modules.php(+0 -4)📝
modules/imap/setup.php(+0 -4)📝
modules/imap/site.js(+3 -49)📝
modules/imap_folders/modules.php(+4 -4)📝
modules/smtp/modules.php(+8 -70)📝
modules/smtp/setup.php(+0 -4)📝
modules/smtp/site.js(+3 -49)📄 Description
This PR removes server forget/save buttons in favor of edit button.
The context is that many fields are being added on server page (lately sieve_tls) and others may come. The forget/save cannot handle these in the future. Edit button instead allows users to edit all together SMTP and IMAP + Sieve with all the fields.
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.