From dc5a51b2154da7b57d0ed3aa3e47b317cbdbdee6 Mon Sep 17 00:00:00 2001
From: daria <dariae9@yahoo.com>
Date: Mon, 17 Mar 2025 14:11:18 +0200
Subject: [PATCH] add error messages

---
 customize.dist/messages.js | 2 ++
 www/admin/inner.js         | 7 ++++++-
 2 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/customize.dist/messages.js b/customize.dist/messages.js
index 469279baf..38206fdbf 100755
--- a/customize.dist/messages.js
+++ b/customize.dist/messages.js
@@ -144,6 +144,8 @@ define(req, function(AppConfig, Default, Language) {
         Messages.admin_addAdminsHint = "Add administrators from their public key or from your contacts list";
         Messages.admin_addAdminsAdd = "Promote a contact to admin";
         Messages.admin_addKeyLabel = "Add an admin using their public key";
+        Messages.admin_errorAddKeyLabel = "Add a valid public key";
+        Messages.admin_errorAddAdminsAdd = "Pick a contact to promote to admin";
 
         Messages.admin_listName = "Admin name";
         Messages.admin_listKey = "Admin key";
diff --git a/www/admin/inner.js b/www/admin/inner.js
index 7c0a157d0..7d53ba7a2 100644
--- a/www/admin/inner.js
+++ b/www/admin/inner.js
@@ -346,7 +346,9 @@ define([
             Util.onClickEnter($keyBtn, () => {
                 let val = $keyInput.val().trim();
                 let key = Keys.canonicalize(val);
-                if (!key) { return; }
+                if (!key) {
+                    UI.warn(Messages.admin_errorAddKeyLabel);
+                    return; }
                 // We have a valid key
                 let name = Messages.admin_admin;
                 try {
@@ -388,6 +390,9 @@ define([
                 let addBtn = blocks.button('primary', 'fa-plus', Messages.tag_add);
                 Util.onClickEnter($(addBtn), () => {
                     var $sel = $(contactsGrid.div).find('.cp-usergrid-user.cp-selected');
+                    if (!$sel.length) {
+                        UI.warn(Messages.admin_errorAddAdminsAdd);
+                        return; }
                     nThen((waitFor) => {
                         $sel.each((i, el) => {
                             const $el = $(el);