From 9ffb2443f6ce012b63114dadc3fc00a75ec5648d Mon Sep 17 00:00:00 2001 From: Ken Sternberg <133134217+kensternberg-authentik@users.noreply.github.com> Date: Tue, 10 Jun 2025 09:49:15 -0700 Subject: [PATCH] web/admin: fix language in certificate import (#14953) * web: Add InvalidationFlow to Radius Provider dialogues ## What - Bugfix: adds the InvalidationFlow to the Radius Provider dialogues - Repairs: `{"invalidation_flow":["This field is required."]}` message, which was *not* propagated to the Notification. - Nitpick: Pretties `?foo=${true}` expressions: `s/\?([^=]+)=\$\{true\}/\1/` ## Note Yes, I know I'm going to have to do more magic when we harmonize the forms, and no, I didn't add the Property Mappings to the wizard, and yes, I know I'm going to have pain with the *new* version of the wizard. But this is a serious bug; you can't make Radius servers with *either* of the current dialogues at the moment. * This (temporary) change is needed to prevent the unit tests from failing. \# What \# Why \# How \# Designs \# Test Steps \# Other Notes * Revert "This (temporary) change is needed to prevent the unit tests from failing." This reverts commit dddde09be571a639ecd041569dd3a282aab3f9be. * web/admin: fix language in certificate import modal There are two buttons on the CertificateKeyPairList: "Create" and "Generate". This language is confusing, as it is unclear what the difference between those two options could be. The second generates and stores a new key pair; the first only creates an entry in our database for an existing pair generated externally. After a short conversation, changing it to "Import" seemed like a safe option. \# What \# Why \# How \# Designs \# Test Steps \# Other Notes --------- Co-authored-by: Jens Langhammer --- web/src/admin/crypto/CertificateKeyPairListPage.ts | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/web/src/admin/crypto/CertificateKeyPairListPage.ts b/web/src/admin/crypto/CertificateKeyPairListPage.ts index 24cbed22cd..940bd0374a 100644 --- a/web/src/admin/crypto/CertificateKeyPairListPage.ts +++ b/web/src/admin/crypto/CertificateKeyPairListPage.ts @@ -214,14 +214,14 @@ export class CertificateKeyPairListPage extends TablePage { renderObjectCreate(): TemplateResult { return html` - ${msg("Create")} - ${msg("Create Certificate-Key Pair")} + ${msg("Import")} + ${msg("Import Existing Certificate-Key Pair")} - + ${msg("Generate")} - ${msg("Generate Certificate-Key Pair")} + ${msg("Generate New Certificate-Key Pair")}