Skip to content

Commit

Permalink
Merge pull request #8693 from Expensify/neil-payment-blink
Browse files Browse the repository at this point in the history
Enable add payment method while payments load
  • Loading branch information
techievivek authored Apr 20, 2022
2 parents 9730c82 + 4db9a9a commit 02af111
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 10 deletions.
5 changes: 0 additions & 5 deletions src/pages/settings/Payments/PaymentMethodList.js
Original file line number Diff line number Diff line change
Expand Up @@ -21,9 +21,6 @@ const propTypes = {
/** What to do when a menu item is pressed */
onPress: PropTypes.func.isRequired,

/** Are we loading payments from the server? */
isLoadingPayments: PropTypes.bool,

/** Is the payment options menu open / active? */
isAddPaymentMenuActive: PropTypes.bool,

Expand Down Expand Up @@ -80,7 +77,6 @@ const defaultProps = {
walletLinkedAccountID: 0,
walletLinkedAccountType: '',
},
isLoadingPayments: false,
isAddPaymentMenuActive: false,
shouldShowAddPaymentMethodButton: true,
filterType: '',
Expand Down Expand Up @@ -164,7 +160,6 @@ class PaymentMethodList extends Component {
icon: Expensicons.Plus,
onPress: e => this.props.onPress(e),
key: 'addPaymentMethodButton',
disabled: this.props.isLoadingPayments,
iconFill: this.props.isAddPaymentMenuActive ? StyleUtils.getIconFillColor(CONST.BUTTON_STATES.PRESSED) : null,
wrapperStyle: this.props.isAddPaymentMenuActive ? [StyleUtils.getButtonBackgroundColorStyle(CONST.BUTTON_STATES.PRESSED)] : [],
});
Expand Down
5 changes: 0 additions & 5 deletions src/pages/settings/Payments/PaymentsPage/BasePaymentsPage.js
Original file line number Diff line number Diff line change
Expand Up @@ -279,7 +279,6 @@ class BasePaymentsPage extends React.Component {
<PaymentMethodList
onPress={this.paymentMethodPressed}
style={[styles.flex4]}
isLoadingPayments={this.props.isLoadingPaymentMethods}
isAddPaymentMenuActive={this.state.shouldShowAddPaymentMenu}
actionPaymentMethodType={this.state.shouldShowDefaultDeleteMenu || this.state.shouldShowPasswordPrompt || this.state.shouldShowConfirmPopover
? this.state.selectedPaymentMethodType
Expand Down Expand Up @@ -440,10 +439,6 @@ export default compose(
walletTransfer: {
key: ONYXKEYS.WALLET_TRANSFER,
},
isLoadingPaymentMethods: {
key: ONYXKEYS.IS_LOADING_PAYMENT_METHODS,
initWithStoredValues: false,
},
userWallet: {
key: ONYXKEYS.USER_WALLET,
},
Expand Down

0 comments on commit 02af111

Please sign in to comment.