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

Sort uninitialized devices in redux state

Browse Source 
Summary: Stops them from jumping around if they try to connect more than once.

Reviewed By: passy

Differential Revision: D13084056

fbshipit-source-id: 803538298c45b9c28e5116f072518e8632b0f1cf
This commit is contained in:
John Knox
2018-11-16 03:36:12 -08:00
committed by Facebook Github Bot
parent 093ab1fe57
commit 7821b6b5a9
1 changed files with 18 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

31
src/reducers/connections.js
View File

@@ -269,31 +269,36 @@ export default function reducer(
...state,
uninitializedClients: state.uninitializedClients
.filter(entry => !isEqual(entry.client, payload))
.concat([{client: payload}]),
.concat([{client: payload}])
.sort((a, b) => a.client.appName.localeCompare(b.client.appName)),
};
}
case 'FINISH_CLIENT_SETUP': {
const {payload} = action;
return {
...state,
uninitializedClients: state.uninitializedClients.map(
c =>
isEqual(c.client, payload.client)
? {...c, deviceId: payload.deviceId}
: c,
),
uninitializedClients: state.uninitializedClients
.map(
c =>
isEqual(c.client, payload.client)
? {...c, deviceId: payload.deviceId}
: c,
)
.sort((a, b) => a.client.appName.localeCompare(b.client.appName)),
};
}
case 'CLIENT_SETUP_ERROR': {
const {payload} = action;
return {
...state,
uninitializedClients: state.uninitializedClients.map(
c =>
isEqual(c.client, payload.client)
? {...c, error: payload.error.message}
: c,
),
uninitializedClients: state.uninitializedClients
.map(
c =>
isEqual(c.client, payload.client)
? {...c, error: payload.error.message}
: c,
)
.sort((a, b) => a.client.appName.localeCompare(b.client.appName)),
};
}
default: