Merge pull request #8647 from Expensify/cmartins-renameOnChange

Rename onChange to onInputChage in AddressSearch

Author: neil-marcellini

src/components/AddressSearch.js

@@ -44,7 +44,7 @@ const propTypes = {
     value: PropTypes.string,
 
     /** A callback function when the value of this field has changed */
-    onChange: PropTypes.func.isRequired,
+    onInputChange: PropTypes.func.isRequired,
 
     /** Customize the TextInput container */
     containerStyles: PropTypes.arrayOf(PropTypes.object),
@@ -113,7 +113,7 @@ const AddressSearch = (props) => {
         if (_.size(values) === 0) {
             return;
         }
-        props.onChange(values);
+        props.onInputChange(values);
     };
 
     return (
@@ -176,7 +176,7 @@ const AddressSearch = (props) => {
                         autoComplete: 'off',
                         onChangeText: (text) => {
                             if (skippedFirstOnChangeTextRef.current) {
-                                props.onChange({street: text});
+                                props.onInputChange({street: text});
                             } else {
                                 skippedFirstOnChangeTextRef.current = true;
                             }

src/pages/EnablePayments/AdditionalDetailsStep.js

@@ -265,7 +265,7 @@ class AdditionalDetailsStep extends React.Component {
                                         label={this.props.translate(this.fieldNameTranslationKeys.addressStreet)}
                                         value={this.props.walletAdditionalDetailsDraft.addressStreet || ''}
                                         containerStyles={[styles.mt4]}
-                                        onChange={(values) => {
+                                        onInputChange={(values) => {
                                             const renamedFields = {
                                                 street: 'addressStreet',
                                                 state: 'addressState',

src/pages/ReimbursementAccount/AddressForm.js

@@ -54,7 +54,7 @@ const AddressForm = props => (
             label={props.translate(props.streetTranslationKey)}
             containerStyles={[styles.mt4]}
             value={props.values.street}
-            onChange={props.onFieldChange}
+            onInputChange={props.onFieldChange}
             errorText={props.errors.street ? props.translate('bankAccount.error.addressStreet') : ''}
         />
         <Text style={[styles.mutedTextLabel, styles.mt1]}>{props.translate('common.noPO')}</Text>

src/pages/settings/Payments/AddDebitCardPage.js

@@ -270,7 +270,7 @@ class DebitCardPage extends Component {
                                 label={this.props.translate('addDebitCardPage.billingAddress')}
                                 containerStyles={[styles.mt4]}
                                 value={this.state.addressStreet}
-                                onChange={(values) => {
+                                onInputChange={(values) => {
                                     const renamedFields = {
                                         street: 'addressStreet',
                                         state: 'addressState',

src/stories/AddressSearch.stories.js

@@ -20,7 +20,7 @@ const Template = (args) => {
     return (
         <AddressSearch
             value={value}
-            onChange={({street}) => setValue(street)}
+            onInputChange={({street}) => setValue(street)}
             // eslint-disable-next-line react/jsx-props-no-spreading
             {...args}
         />