Fix Flipper lints #5

Summary: A few more lints. Some already got fixed since the task was created.

Reviewed By: timur-valiev

Differential Revision: D30782192

fbshipit-source-id: 2d03d24034173d632280035e6ba68f485fbe62f3

desktop/app/src/sandy-chrome/SandyApp.tsx

@@ -103,25 +103,27 @@ export function SandyApp() {
 
   useEffect(() => {
     if (fbConfig.warnFBEmployees && isProduction()) {
-      isFBEmployee().then((isEmployee) => {
+      isFBEmployee()
-        if (isEmployee) {
+        .then((isEmployee) => {
-          notification.warning({
+          if (isEmployee) {
-            placement: 'bottomLeft',
+            notification.warning({
-            message: 'Please use Flipper@FB',
+              placement: 'bottomLeft',
-            description: (
+              message: 'Please use Flipper@FB',
-              <>
+              description: (
-                You are using the open-source version of Flipper. Install the
+                <>
-                internal build from{' '}
+                  You are using the open-source version of Flipper. Install the
-                <Link href="munki://detail-Flipper">
+                  internal build from{' '}
-                  Managed Software Center
+                  <Link href="munki://detail-Flipper">
-                </Link>{' '}
+                    Managed Software Center
-                to get access to more plugins.
+                  </Link>{' '}
-              </>
+                  to get access to more plugins.
-            ),
+                </>
-            duration: null,
+              ),
-          });
+              duration: null,
-        }
+            });
-      });
+          }
+        })
+        .catch((e) => console.error('isEmployee check failed:', e));
     }
   }, []);
 

desktop/app/src/sandy-chrome/appinspect/AppSelector.tsx

@@ -186,6 +186,8 @@ function computeEntries(
         // hide non default devices, unless they have a connected client or plugins
         canBeDefaultDevice(device) ||
         device.hasDevicePlugins ||
+        // Deliberate use of Sync.
+        // eslint-disable-next-line node/no-sync
         clients.some((c) => c.deviceSync === device),
     )
     .map((device) => {

desktop/app/src/sandy-chrome/notification/Notification.tsx

@@ -217,7 +217,8 @@ export function Notification() {
       activeNotifications.map((noti) => {
         const client = getClientById(store, noti.client);
         const device = client
-          ? client.deviceSync
+          ? // eslint-disable-next-line node/no-sync
+            client.deviceSync
           : getDeviceById(store, noti.client);
         const plugin = getPlugin(noti.pluginId);
         return {
@@ -311,6 +312,7 @@ export function openNotification(store: Store, noti: PluginNotificationOrig) {
       selectPlugin({
         selectedPlugin: noti.pluginId,
         selectedApp: noti.client,
+        // eslint-disable-next-line node/no-sync
         selectedDevice: client.deviceSync,
         deepLinkPayload: noti.notification.action,
       }),