From f741c62f590d098c2ced087e00efde4d8dd850b4 Mon Sep 17 00:00:00 2001 From: Stanislav Levin Date: Sun, 16 Sep 2018 23:14:45 +0300 Subject: [PATCH] Add a title to 'remove' dialog for details of 'OTP Tokens' entity To improve translation quality the title of 'Remove' dialog, which is initialized within details of the entity, should be specified explicitly in the spec and should be an entire sentence. Fixes: https://pagure.io/freeipa/issue/7702 Reviewed-By: Serhii Tsymbaliuk --- install/ui/src/freeipa/otptoken.js | 3 ++- ipaserver/plugins/internal.py | 3 +++ 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/install/ui/src/freeipa/otptoken.js b/install/ui/src/freeipa/otptoken.js index ebd161dc0..95bb8808e 100644 --- a/install/ui/src/freeipa/otptoken.js +++ b/install/ui/src/freeipa/otptoken.js @@ -226,7 +226,8 @@ return { $type: 'association', name: 'managedby_user', add_method: 'add_managedby', - remove_method: 'remove_managedby' + remove_method: 'remove_managedby', + remove_title: '@i18n:objects.otptoken.remove_users_managing', } ], diff --git a/ipaserver/plugins/internal.py b/ipaserver/plugins/internal.py index 6b2d59236..f7fbac726 100644 --- a/ipaserver/plugins/internal.py +++ b/ipaserver/plugins/internal.py @@ -833,6 +833,9 @@ class i18n_messages(Command): "disable": _("Disable token"), "enable": _("Enable token"), "remove": _("Remove OTP tokens"), + "remove_users_managing": _( + "Remove users managing OTP token '${primary_key}'" + ), "show_qr": _("Show QR code"), "show_uri": _("Show configuration uri"), "type_hotp": _("Counter-based (HOTP)"),