jens/flipper
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Better notification messaging

Browse Source 
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
This commit is contained in:
Lorenzo Blasa
2022-02-16 03:30:46 -08:00
committed by Facebook GitHub Bot
parent 9d2f11f505
commit 77ab722b52
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
desktop/flipper-ui-core/src/dispatcher/flipperServer.tsx
View File

@@ -46,7 +46,7 @@ export function connectFlipperServerToStore(
server.on('server-error', (err) => { server.on('server-error', (err) => {
notification.error({ notification.error({
message: 'Failed to start connection server', message: 'Connection error',
description: description:
err.code === 'EADDRINUSE' ? ( err.code === 'EADDRINUSE' ? (
<> <>
@@ -62,7 +62,7 @@ export function connectFlipperServerToStore(
{'' + err} {'' + err}
</> </>
) : ( ) : (
<>Failed to start Flipper server: {err.message ?? err}</> <>{err.message ?? err}</>
), ),
duration: null, duration: null,
}); });