Fix: [TextFile] do not pass onChange when input prop is provided

[nicosampler]

No description provided.

[ghost]

Travis automatic deployment:
https://pr43--safereactcomponents.review.gnosisdev.com

[mmv08]

is it possible to reflect types accordingly? so if input is defined, onChange shouldn't be

https://stackoverflow.com/questions/56437759/typescript-utility-type-for-conditional-props-based-on-entered-value-of-other-p
https://stackoverflow.com/questions/54587188/typescript-require-the-property-if-other-is-defined-requiredif

[nicosampler]

@mikheevm I'm trying something like this:

type ConditionalProps =
  | {
      input: React.InputHTMLAttributes<HTMLInputElement>; // added for compatibility with react-final-form
      onChange?: (event: React.ChangeEvent<HTMLInputElement>) => void;
    }
  | {
      onChange: (event: React.ChangeEvent<HTMLInputElement>) => void;
    };

interface BaseProps extends React.InputHTMLAttributes<HTMLInputElement> {
  value: string;
  label: string;
  readOnly?: boolean;
  meta?: {
    error?: string;
  };
  startAdornment?: React.ReactElement;
  endAdornment?: React.ReactElement;
  className?: string;
}

type Props = BaseProps & ConditionalProps;

but it's not working, seems like we need to replicate all the keys. One with all keys but input and the other with all the types but onChange. Seems like overkill to me. wdyt?