From 77ab722b524fdfc882d00eb57c787db1efd26203 Mon Sep 17 00:00:00 2001 From: Lorenzo Blasa Date: Wed, 16 Feb 2022 03:30:46 -0800 Subject: [PATCH] Better notification messaging Summary: 'Failed to start connection server' is a bit misleading. 'server-error' is used for both server and client errors. Changelog: Better notification messaging Reviewed By: aigoncharov Differential Revision: D34269045 fbshipit-source-id: f51a28fc0e9ba394b464e5bfca4d0e497f740b1b --- desktop/flipper-ui-core/src/dispatcher/flipperServer.tsx | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/desktop/flipper-ui-core/src/dispatcher/flipperServer.tsx b/desktop/flipper-ui-core/src/dispatcher/flipperServer.tsx index c10db57e8..f07c8c901 100644 --- a/desktop/flipper-ui-core/src/dispatcher/flipperServer.tsx +++ b/desktop/flipper-ui-core/src/dispatcher/flipperServer.tsx @@ -46,7 +46,7 @@ export function connectFlipperServerToStore( server.on('server-error', (err) => { notification.error({ - message: 'Failed to start connection server', + message: 'Connection error', description: err.code === 'EADDRINUSE' ? ( <> @@ -62,7 +62,7 @@ export function connectFlipperServerToStore( {'' + err} ) : ( - <>Failed to start Flipper server: {err.message ?? err} + <>{err.message ?? err} ), duration: null, });