[Cleanup] Address feedback

src/lib/AppRegistry.tsx

@@ -17,16 +17,11 @@ import HomeScene from "./Scenes/Home"
 import renderWithLoadProgress from "./utils/renderWithLoadProgress"
 import { Schema, screenTrack as track } from "./utils/track"
 
-// FIXME: This isnt being used
-// Analytics wrapper for all of our top level React components
-// function AddTrack(_pageName: string) {
-//   return component => component
-// }
-
 interface ArtistProps {
   artistID: string
   isPad: boolean
 }
+
 const Artist: React.SFC<ArtistProps> = track<ArtistProps>(props => {
   return {
     context_screen: Schema.PageNames.ArtistPage,
@@ -35,7 +30,7 @@ const Artist: React.SFC<ArtistProps> = track<ArtistProps>(props => {
   }
 })(props => <ArtistRenderer {...props} render={renderWithLoadProgress(Containers.Artist, props)} />)
 
-const Inbox: React.SFC<{}> = track<{}>(_props => {
+const Inbox: React.SFC<{}> = track<{}>(() => {
   return { context_screen: Schema.PageNames.InboxPage, context_screen_owner_type: null }
 })(() => <InboxRenderer render={renderWithLoadProgress(Containers.Inbox)} />)
 

src/lib/Components/Artist/Articles/Article.tsx

@@ -7,20 +7,9 @@ import fonts from "lib/data/fonts"
 import SwitchBoard from "lib/NativeModules/SwitchBoard"
 import ImageView from "../../OpaqueImageView"
 
-interface Props extends ViewProperties {
-  article: {
-    href: string
-    author: {
-      name
-    }
-    thumbnail_image: {
-      url
-    }
-    thumbnail_title: string
-  }
-}
+interface Props extends RelayProps, ViewProperties {}
 
-class Article extends React.Component<Props, {}> {
+class Article extends React.Component<Props> {
   handleTap() {
     SwitchBoard.presentNavigationViewController(this, this.props.article.href)
   }
@@ -101,16 +90,15 @@ export default createFragmentContainer(
   `
 )
 
-// FIXME: This is being unused in code
-// interface RelayProps {
-//   article: {
-//     thumbnail_title: string | null
-//     href: string | null
-//     author: {
-//       name: string | null
-//     } | null
-//     thumbnail_image: {
-//       url: string | null
-//     } | null
-//   }
-// }
+interface RelayProps {
+  article: {
+    thumbnail_title: string | null
+    href: string | null
+    author: {
+      name: string | null
+    } | null
+    thumbnail_image: {
+      url: string | null
+    } | null
+  }
+}

src/lib/Components/Artist/Articles/index.tsx

@@ -5,11 +5,11 @@ import { createFragmentContainer, graphql } from "react-relay"
 import SerifText from "../../Text/Serif"
 import Article from "./Article"
 
-interface Props extends ViewProperties {
+interface Props extends RelayProps, ViewProperties {
   articles: any[]
 }
 
-class Articles extends React.Component<Props, {}> {
+class Articles extends React.Component<Props> {
   render() {
     const articles = this.props.articles
     return (
@@ -49,9 +49,8 @@ export default createFragmentContainer(
   `
 )
 
-// FIXME: This is being unused in code
-// interface RelayProps {
-//   articles: Array<{
-//     __id: string
-//   } | null> | null
-// }
+interface RelayProps {
+  articles: Array<{
+    __id: string
+  } | null> | null
+}

src/lib/Components/Artist/Artworks/index.tsx

@@ -19,15 +19,7 @@ interface RenderSectionParams {
   mapPropsToArtworksConnection: (Props) => any
 }
 
-interface Props extends ViewProperties {
-  artist: {
-    counts: {
-      for_sale_artworks: number
-      artworks: number
-    }
-    not_for_sale_artworks: any[]
-    for_sale_artworks: any[]
-  }
+interface Props extends RelayProps, ViewProperties {
   relay: any
 }
 
@@ -138,14 +130,13 @@ export default createFragmentContainer(
   `
 )
 
-// FIXME: This is unused in code
-// interface RelayProps {
-//   artist: {
-//     counts: {
-//       artworks: boolean | number | string | null
-//       for_sale_artworks: boolean | number | string | null
-//     } | null
-//     for_sale_artworks: Array<boolean | number | string | null> | null
-//     not_for_sale_artworks: Array<boolean | number | string | null> | null
-//   }
-// }
+interface RelayProps {
+  artist: {
+    counts: {
+      artworks: number | null
+      for_sale_artworks: number | null
+    } | null
+    for_sale_artworks: Array<number | null> | null
+    not_for_sale_artworks: Array<boolean | number | string | null> | null
+  }
+}

src/lib/Components/Artist/Biography.tsx

@@ -9,9 +9,7 @@ import SerifText from "../Text/Serif"
 
 const sideMargin = Dimensions.get("window").width > 700 ? 50 : 0
 
-interface Props extends ViewProperties {
-  artist: any
-}
+interface Props extends RelayProps, ViewProperties {}
 
 class Biography extends React.Component<Props, any> {
   render() {
@@ -70,10 +68,9 @@ export default createFragmentContainer(
   `
 )
 
-// FIXME: This isn't being used in code
-// interface RelayProps {
-//   artist: {
-//     bio: string | null
-//     blurb: string | null
-//   }
-// }
+interface RelayProps {
+  artist: {
+    bio: string | null
+    blurb: string | null
+  }
+}

src/lib/Components/Artist/Shows/Metadata.tsx

@@ -7,21 +7,7 @@ import SerifText from "lib/Components/Text/Serif"
 import colors from "lib/data/colors"
 import fonts from "lib/data/fonts"
 
-interface Props extends ViewProperties {
-  show: {
-    name: string
-    kind: string
-    exhibition_period: string
-    location?: {
-      city: string
-    }
-    status_update?: string
-    status: string
-    partner?: {
-      name: string
-    }
-  }
-}
+interface Props extends RelayProps, ViewProperties {}
 
 class Metadata extends React.Component<Props, any> {
   render() {
@@ -120,19 +106,18 @@ export default createFragmentContainer(
   `
 )
 
-// FIXME: This isn't being used in code
-// interface RelayProps {
-//   show: {
-//     kind: string | null
-//     name: string | null
-//     exhibition_period: string | null
-//     status_update: string | null
-//     status: string | null
-//     partner: {
-//       name: string | null
-//     } | null
-//     location: {
-//       city: string | null
-//     } | null
-//   }
-// }
+interface RelayProps {
+  show: {
+    kind: string | null
+    name: string | null
+    exhibition_period: string | null
+    status_update: string | null
+    status: string | null
+    partner: {
+      name: string | null
+    } | null
+    location: {
+      city: string | null
+    } | null
+  }
+}

src/lib/Components/Artist/Shows/Show.tsx

@@ -6,13 +6,7 @@ import SwitchBoard from "../../../NativeModules/SwitchBoard"
 import OpaqueImageView from "../../OpaqueImageView"
 import Metadata from "./Metadata"
 
-interface Props extends ViewProperties {
-  show: {
-    href: string
-    cover_image: {
-      url: string
-    }
-  }
+interface Props extends RelayProps, ViewProperties {
   styles?: {
     container?: any
     image?: any
@@ -56,12 +50,11 @@ export default createFragmentContainer(
   `
 )
 
-// FIXME: This isn't being used in code
-// interface RelayProps {
-//   show: {
-//     href: string | null
-//     cover_image: {
-//       url: string | null
-//     } | null
-//   }
-// }
+interface RelayProps {
+  show: {
+    href: string | null
+    cover_image: {
+      url: string | null
+    } | null
+  }
+}

src/lib/Components/Artist/Shows/SmallList.tsx

@@ -6,9 +6,7 @@ import Show from "./Show"
 
 import colors from "lib/data/colors"
 
-interface Props extends ViewProperties {
-  shows: any[]
-}
+interface Props extends RelayProps, ViewProperties {}
 
 interface State {
   dataSource: ListViewDataSource
@@ -72,7 +70,6 @@ export default createFragmentContainer(
   `
 )
 
-// FIXME: This isn't being used in code
-// interface RelayProps {
-//   shows: Array<{} | null> | null
-// }
+interface RelayProps {
+  shows: Array<{} | null> | null
+}

src/lib/Components/Artist/Shows/VariableSizeShowsList.tsx

@@ -5,7 +5,7 @@ import { LayoutChangeEvent, StyleSheet, View, ViewProperties, ViewStyle } from "
 
 import Show from "./Show"
 
-interface Props extends ViewProperties {
+interface Props extends RelayProps, ViewProperties {
   showSize: "medium" | "large"
   shows: any[]
 }
@@ -102,9 +102,8 @@ export default createFragmentContainer(
   `
 )
 
-// FIXME: This isn't being used in code
-// interface RelayProps {
-//   shows: Array<{
-//     __id: string
-//   } | null> | null
-// }
+interface RelayProps {
+  shows: Array<{
+    __id: string
+  } | null> | null
+}

src/lib/Components/Consignments/Screens/Overview.tsx

@@ -180,9 +180,6 @@ export default class Info extends React.Component<Props, State> {
 
     const isPad = Dimensions.get("window").width > 700
 
-    // TODO: Reenable when used
-    // const isPadHorizontal = Dimensions.get("window").height > 700
-
     return (
       <ConsignmentBG showCloseButton>
         <ScrollView style={{ flex: 1 }} alwaysBounceVertical={false} centerContent>

src/lib/Components/Consignments/Screens/Welcome.tsx

@@ -35,9 +35,6 @@ export default class Welcome extends React.Component<Props, null> {
   render() {
     const isPad = Dimensions.get("window").width > 700
 
-    // TODO: Reenable when used
-    // const isPadHorizontal = Dimensions.get("window").height > 700
-
     const TOS = () =>
       <Text style={{ textDecorationStyle: "solid", textDecorationLine: "underline" }} onPress={this.TOSTapped}>
         Terms of Sevice

src/lib/Components/Consignments/Submission/geminiUploadToS3.ts

@@ -96,12 +96,9 @@ export const uploadFileToS3 = async (file: string, req: GeminiCredsInput, res: G
     const bucket = asset.policy_document.conditions.bucket
     const uploadURL = `https://${bucket}.s3.amazonaws.com`
 
-    // FIXME: This is being unused in 103; does this uploader work?
-    // const filename = file.split(/\//g).pop()
-
     const data = {
       "Content-Type": "image/jpg",
-      key: geminiKey + "/${filename}",
+      key: geminiKey + "/${filename}", // NOTE: This form (which _looks_ like ES6 interpolation) is required by AWS
       AWSAccessKeyId: asset.credentials,
       acl: req.acl,
       success_action_status: asset.policy_document.conditions.success_action_status,

src/lib/Components/Gene/About.tsx

@@ -8,7 +8,7 @@ import Biography from "./Biography"
 import RelatedArtists from "../RelatedArtists"
 import Separator from "../Separator"
 
-class About extends React.Component<RelayPropsWorkaround, any> {
+class About extends React.Component<RelayProps, any> {
   render() {
     return (
       <View>
@@ -56,15 +56,8 @@ export default createFragmentContainer(
   `
 )
 
-// FIXME: This isn't being used in code
-// interface RelayProps {
-//   gene: {
-//     trending_artists: Array<boolean | number | string | null> | null
-//   }
-// }
-
-interface RelayPropsWorkaround {
+interface RelayProps {
   gene: {
-    trending_artists: any[]
+    trending_artists: Array<boolean | number | string | null> | null
   }
 }

src/lib/Components/Gene/__tests__/About-tests.tsx

@@ -42,6 +42,6 @@ it("shows trending artists correctly", () => {
     ],
   }
 
-  const about = renderer.create(<About gene={gene} />).toJSON()
+  const about = renderer.create(<About gene={gene as any} />).toJSON()
   expect(about).toMatchSnapshot()
 })

src/lib/Components/Home/ArtworkRails/ArtworkRail.tsx

@@ -67,7 +67,9 @@ export class ArtworkRail extends React.Component<Props & RelayPropsWorkaround, S
   componentDidMount() {
     if (this.props.relay) {
       this.props.relay.refetch({ __id: this.props.rail.__id, fetchContent: true }, null, error => {
-        console.error("ArtworkRail.tsx |", error)
+        if (error) {
+          console.error("ArtworkRail.tsx |", error)
+        }
 
         this.setState({
           didPerformFetch: true,