From 82537967435db2dcd298a03ee5fd32e6bf38ebc7 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Aur=C3=A9lien=20Sibiril?= <81782+aureliensibiril@users.noreply.github.com> Date: Thu, 4 Jun 2026 19:09:04 +0200 Subject: [PATCH] Use Field onValueChange in the Zendesk dialog MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Field exposes onValueChange(value) and wires the input's onChange itself, so the explicit event-typed onChange handler was redundant. Signed-off-by: Aurélien Sibiril <81782+aureliensibiril@users.noreply.github.com> --- .../access-reviews/dialogs/AddAccessSourceDialog.tsx | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/apps/console/src/pages/organizations/access-reviews/dialogs/AddAccessSourceDialog.tsx b/apps/console/src/pages/organizations/access-reviews/dialogs/AddAccessSourceDialog.tsx index 9c193a778..2f2c1bfa1 100644 --- a/apps/console/src/pages/organizations/access-reviews/dialogs/AddAccessSourceDialog.tsx +++ b/apps/console/src/pages/organizations/access-reviews/dialogs/AddAccessSourceDialog.tsx @@ -857,8 +857,7 @@ export function AddAccessSourceDialog({ label={__("Zendesk subdomain")} placeholder={__("acme")} value={zendeskSubdomain} - onChange={(e: React.ChangeEvent) => - setZendeskSubdomain(e.target.value)} + onValueChange={setZendeskSubdomain} help={__("The part of .zendesk.com")} required autoFocus