[PR #2964] [MERGED] Better UX on failling to save parameters #2844

Closed
opened 2026-02-27 12:13:01 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/modoboa/modoboa/pull/2964
Author: @Spitfireap
Created: 4/23/2023
Status: Merged
Merged: 4/25/2023
Merged by: @tonioo

Base: masterHead: frontend-parameters-qol


📝 Commits (4)

📊 Changes

22 files changed (+1551 additions, -1306 deletions)

View changed files

📝 .gitignore (+1 -1)
📝 MANIFEST.in (+1 -0)
📝 frontend/locale/br/LC_MESSAGES/app.po (+78 -61)
📝 frontend/locale/cs/LC_MESSAGES/app.po (+78 -61)
📝 frontend/locale/de/LC_MESSAGES/app.po (+78 -61)
📝 frontend/locale/el/LC_MESSAGES/app.po (+78 -61)
📝 frontend/locale/en/LC_MESSAGES/app.po (+78 -61)
📝 frontend/locale/es/LC_MESSAGES/app.po (+78 -61)
📝 frontend/locale/fi/LC_MESSAGES/app.po (+78 -61)
📝 frontend/locale/fr/LC_MESSAGES/app.po (+129 -206)
📝 frontend/locale/it/LC_MESSAGES/app.po (+78 -61)
📝 frontend/locale/ja/LC_MESSAGES/app.po (+78 -61)
📝 frontend/locale/nl/LC_MESSAGES/app.po (+78 -61)
📝 frontend/locale/pl/LC_MESSAGES/app.po (+78 -61)
📝 frontend/locale/po/LC_MESSAGES/app.po (+78 -61)
📝 frontend/locale/pt-br/LC_MESSAGES/app.po (+78 -61)
📝 frontend/locale/pt/LC_MESSAGES/app.po (+78 -61)
📝 frontend/locale/ru/LC_MESSAGES/app.po (+78 -61)
📝 frontend/locale/sv/LC_MESSAGES/app.po (+78 -61)
📝 frontend/locale/tr/LC_MESSAGES/app.po (+78 -61)

...and 2 more files

📄 Description

  • Added a notification when parameters fail to save and highlighted tabs with parameters containing errors in red:
    image
  • Updated translation

🔄 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/modoboa/modoboa/pull/2964 **Author:** [@Spitfireap](https://github.com/Spitfireap) **Created:** 4/23/2023 **Status:** ✅ Merged **Merged:** 4/25/2023 **Merged by:** [@tonioo](https://github.com/tonioo) **Base:** `master` ← **Head:** `frontend-parameters-qol` --- ### 📝 Commits (4) - [`17bbd56`](https://github.com/modoboa/modoboa/commit/17bbd566a20fd3bae8766182dee4b7152c188a17) Better UX on failling to save parameters - [`d3b7dbd`](https://github.com/modoboa/modoboa/commit/d3b7dbd53ac3ffa825ff5e65661cd5b141a7a72a) Code cleaning - [`2f1d736`](https://github.com/modoboa/modoboa/commit/2f1d736097c9a49c970755562b0d768e7c240c9b) Added favicon for new-admin on build - [`dfd2ea1`](https://github.com/modoboa/modoboa/commit/dfd2ea1b61b9cca936c27757bf36a0f295ce06c4) cleaning ### 📊 Changes **22 files changed** (+1551 additions, -1306 deletions) <details> <summary>View changed files</summary> 📝 `.gitignore` (+1 -1) 📝 `MANIFEST.in` (+1 -0) 📝 `frontend/locale/br/LC_MESSAGES/app.po` (+78 -61) 📝 `frontend/locale/cs/LC_MESSAGES/app.po` (+78 -61) 📝 `frontend/locale/de/LC_MESSAGES/app.po` (+78 -61) 📝 `frontend/locale/el/LC_MESSAGES/app.po` (+78 -61) 📝 `frontend/locale/en/LC_MESSAGES/app.po` (+78 -61) 📝 `frontend/locale/es/LC_MESSAGES/app.po` (+78 -61) 📝 `frontend/locale/fi/LC_MESSAGES/app.po` (+78 -61) 📝 `frontend/locale/fr/LC_MESSAGES/app.po` (+129 -206) 📝 `frontend/locale/it/LC_MESSAGES/app.po` (+78 -61) 📝 `frontend/locale/ja/LC_MESSAGES/app.po` (+78 -61) 📝 `frontend/locale/nl/LC_MESSAGES/app.po` (+78 -61) 📝 `frontend/locale/pl/LC_MESSAGES/app.po` (+78 -61) 📝 `frontend/locale/po/LC_MESSAGES/app.po` (+78 -61) 📝 `frontend/locale/pt-br/LC_MESSAGES/app.po` (+78 -61) 📝 `frontend/locale/pt/LC_MESSAGES/app.po` (+78 -61) 📝 `frontend/locale/ru/LC_MESSAGES/app.po` (+78 -61) 📝 `frontend/locale/sv/LC_MESSAGES/app.po` (+78 -61) 📝 `frontend/locale/tr/LC_MESSAGES/app.po` (+78 -61) _...and 2 more files_ </details> ### 📄 Description * Added a notification when parameters fail to save and highlighted tabs with parameters containing errors in red: ![image](https://user-images.githubusercontent.com/45575529/233853775-eb42453d-b2f6-4d55-8572-458114bb1c60.png) * Updated translation --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-27 12:13:01 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/modoboa-modoboa#2844
No description provided.