chore: Remove/Convert a couple files to TS, shared utils Part 2

src/pages/AccountSettings/tabs/Admin/ManageAdminCard/AddAdmins/AddAdmins.jsx

@@ -9,7 +9,7 @@ import useClickAway from 'react-use/lib/useClickAway'
 import User from 'old_ui/User'
 import { ApiFilterEnum } from 'services/navigation'
 import { useUsers } from 'services/users'
-import { getOwnerImg } from 'shared/utils'
+import { getOwnerImg } from 'shared/utils/ownerHelpers'
 import TextInput from 'ui/TextInput'
 
 const styles = {

src/pages/AccountSettings/tabs/Admin/ManageAdminCard/AdminTable/AdminTable.tsx

@@ -10,7 +10,7 @@ import { useInView } from 'react-intersection-observer'
 import { useParams } from 'react-router-dom'
 
 import { useInfiniteUsers, useUpdateUser } from 'services/users'
-import { getOwnerImg } from 'shared/utils'
+import { getOwnerImg } from 'shared/utils/ownerHelpers'
 import Avatar, { DefaultAuthor } from 'ui/Avatar'
 import Button from 'ui/Button'
 import Icon from 'ui/Icon'

src/pages/MembersPage/MembersList/MembersTable/MembersTable.tsx

@@ -12,8 +12,8 @@ import { useParams } from 'react-router-dom'
 
 import { useAccountDetails } from 'services/account'
 import { Member, useInfiniteUsers } from 'services/users'
-import { getOwnerImg } from 'shared/utils'
 import { isFreePlan } from 'shared/utils/billing'
+import { getOwnerImg } from 'shared/utils/ownerHelpers'
 import Avatar, { DefaultAuthor } from 'ui/Avatar'
 import Icon from 'ui/Icon'
 import Spinner from 'ui/Spinner'

src/pages/RepoPage/CoverageTab/OverviewTab/hooks/useRepoCoverageTimeseries.test.tsx

@@ -4,7 +4,7 @@ import { graphql, HttpResponse } from 'msw'
 import { setupServer } from 'msw/node'
 import { MemoryRouter, Route } from 'react-router-dom'
 
-import { Trend } from 'shared/utils/legacyCharts'
+import { Trend } from 'shared/utils/timeseriesCharts'
 
 import { useRepoCoverageTimeseries } from './useRepoCoverageTimeseries'
 

src/services/account/useUpgradePlan.ts

@@ -33,7 +33,7 @@ export function useUpgradePlan({ provider, owner }: useUpgradePlanParams) {
         },
       }
       return Api.patch({ path, provider, body }).then((data) => {
-        if (data.checkoutSessionId) {
+        if (data?.checkoutSessionId) {
           // redirect to stripe checkout if there is a checkout session id
           return redirectToStripe(data.checkoutSessionId)
         }

src/shared/utils/camelizeKeys.test.js → src/shared/utils/camelizeKeys.test.ts

@@ -17,7 +17,9 @@ describe('camelizeKeys', () => {
 
   it('else passes through', () => {
     expect(camelizeKeys([1, 2, 3])).toStrictEqual([1, 2, 3])
+    // @ts-expect-error
     expect(camelizeKeys(1)).toStrictEqual(1)
+    // @ts-expect-error
     expect(camelizeKeys('test')).toStrictEqual('test')
     expect(camelizeKeys()).toStrictEqual({})
   })

src/shared/utils/camelizeKeys.js → src/shared/utils/camelizeKeys.ts

@@ -1,6 +1,8 @@
 import camelCase from 'lodash/camelCase'
 
-export function camelizeKeys(obj = {}) {
+export function camelizeKeys(
+  obj: Record<string, any> = {}
+): Record<string, any> {
   if (Array.isArray(obj)) {
     return obj.map((v) => camelizeKeys(v))
   } else if (obj !== null && obj.constructor === Object) {

src/shared/utils/index.js

@@ -1,3 +1 @@
-export * from './ownerHelpers'
 export * from './provider'
-export * from './exceptions'