fix(FeedTransformation): Revalidate errors if user selects a table for transform.

Author: binh-dam-ibigroup

lib/manager/components/transform/FeedTransformation.js

@@ -40,6 +40,8 @@ const transformationTypes = {
   }
 }
 
+const TABLE_MUST_BE_DEFINED_ERROR = 'Table must be defined'
+
 /**
  * Component that renders fields for one feed transformation
  * (e.g., ReplaceFileFromStringTransformation).
@@ -49,6 +51,15 @@ export default class FeedTransformation extends Component<Props, {errors: Array<
     errors: []
   }
 
+  componentDidUpdate (prevProps: Props) {
+    // Update error state if props.transformation.table changed
+    // and 'Table must be defined' no longer applies.
+    const { table } = this.props.transformation
+    if (table && prevProps.transformation.table !== table) {
+      this.setState({ errors: this.state.errors.filter(e => e !== TABLE_MUST_BE_DEFINED_ERROR) })
+    }
+  }
+
   _getFieldsForType = (type: string) => {
     const {feedSource, index, onChange, transformation} = this.props
     const FieldsComponent = transformationTypes[type].component
@@ -66,7 +77,7 @@ export default class FeedTransformation extends Component<Props, {errors: Array<
   _onValidationErrors = (errors: Array<string>) => {
     const issues: Array<string> = []
     if (!this.props.transformation.table) {
-      issues.push('Table must be defined')
+      issues.push(TABLE_MUST_BE_DEFINED_ERROR)
     }
     this.setState({ errors: issues.concat(errors) })
   }