Lock 'react' and 'react-dom' to '^16.14.0' version and fix some types (#152)

Author: usenko-timur

.flowconfig

@@ -11,8 +11,6 @@
 [lints]
 
 [options]
-module.file_ext=.ts
-module.file_ext=.tsx
 suppress_comment= \\(.\\|\n\\)*\\$FlowFixMe
 
 [strict]

CHANGELOG.md

@@ -11,6 +11,8 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 
 - Update prettier and eslint configurations;
 
+- Lock `react` and `react-dom` to `^16.14.0` version;
+
 ## [7.10.0]
 
 - Pass menu subitems to filtering

package-lock.json

@@ -37,8 +37,8 @@
     "promise": "^8.0.1",
     "raf": "^3.4.0",
     "ramda": "^0.25.0",
-    "react": "^17.0.2",
-    "react-dom": "^17.0.2",
+    "react": "^16.14.0",
+    "react-dom": "^16.14.0",
     "react-redux": "^5.0.7",
     "react-router": "^4.3.1",
     "react-router-dom": "^4.3.1",
@@ -54,7 +54,7 @@
     "@babel/plugin-proposal-decorators": "^7.14.2",
     "@babel/preset-flow": "^7.13.13",
     "@emotion/babel-plugin": "^11.3.0",
-    "@tarantool.io/eslint-config": "^1.0.3",
+    "@tarantool.io/eslint-config": "^1.0.4",
     "@tarantool.io/import-sort-style": "^1.0.3",
     "@tarantool.io/lua-bundler-webpack-plugin": "^1.0.1",
     "@tarantool.io/prettier-config": "^1.0.3",
@@ -85,7 +85,7 @@
     "postcss-flexbugs-fixes": "^3.2.0",
     "postcss-loader": "^3.0.0",
     "react-dev-utils": "^11.0.4",
-    "react-test-renderer": "16.14.0",
+    "react-test-renderer": "^16.14.0",
     "style-loader": "^1.0.0",
     "typescript": "4.4.2",
     "url-loader": "^2.1.0",

package.json

@@ -4,6 +4,7 @@ import { ApolloClient } from 'apollo-client';
 import { from } from 'apollo-link';
 import { HttpLink } from 'apollo-link-http';
 import axios from 'axios';
+// eslint-disable-next-line import/no-named-as-default
 import gql from 'graphql-tag';
 import nock from 'nock';
 

src/api.test.js

@@ -1,5 +1,4 @@
 /* eslint-disable sonarjs/no-duplicate-string */
-// @flow
 import React from 'react';
 import { Provider } from 'react-redux';
 import { Route, Router, Switch } from 'react-router-dom';
@@ -65,7 +64,7 @@ test('not found', () => {
   }
 
   class RouteProvider extends React.Component {
-    render(): React.ReactNode {
+    render() {
       const { store, core } = this.props;
       return (
         <div>

src/components/Routes.test.js

@@ -7,6 +7,7 @@ const fnSubstituteAction = [PAGE_FILTER_ADD, PAGE_FILTER_REMOVE];
 
 export default () => (next: Function) => (action: FSA) => {
   if (fnSubstituteAction.includes(action.type)) {
+    // $FlowFixMe
     action.payload.fn = disposableFunctionKey(action.payload.fn);
     if (action.type === PAGE_FILTER_REMOVE) {
       disposeFnByKey(action.payload.fn);

src/store/middlewares/pageFilter.js

@@ -8,18 +8,18 @@ const fnKeyMap = new Map<Function, string>();
 fnMap['1'] = 1;
 delete fnMap['1'];
 
-export const getKeyByFn: (Function) => ?string = (fn) => fnKeyMap.get(fn);
+export const getKeyByFn = (fn: ?Function): ?string => (fn ? fnKeyMap.get(fn) : undefined);
 
-export const disposableFunctionKey: (Function) => string = (fn) => {
+export const disposableFunctionKey = (fn: Function): string => {
   const key = getKeyByFn(fn) || nanoid();
   fnMap[key] = fn;
   fnKeyMap.set(fn, key);
   return key;
 };
 
-export const getFnByKey: (string) => Function = (key) => fnMap[key];
+export const getFnByKey = (key: string): ?Function => fnMap[key];
 
-export const disposeFnByKey: (string) => void = (key) => {
+export const disposeFnByKey = (key: string): void => {
   const fn = fnMap[key];
   delete fnMap[key];
   fnKeyMap.delete(fn);