Skip to content

Commit

Permalink
Merge pull request #8882 from Expensify/afonseca_remove_confimation_w…
Browse files Browse the repository at this point in the history
…orkspace

Commenting out the confimation modal to test something in staging.

(cherry picked from commit c0f2264)
  • Loading branch information
tgolen authored and OSBotify committed May 6, 2022
1 parent 5dc184f commit c132c97
Showing 1 changed file with 10 additions and 5 deletions.
15 changes: 10 additions & 5 deletions src/pages/workspace/WorkspaceInitialPage.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,8 @@ import ROUTES from '../../ROUTES';
import styles from '../../styles/styles';
import Tooltip from '../../components/Tooltip';
import Text from '../../components/Text';
import ConfirmModal from '../../components/ConfirmModal';

// import ConfirmModal from '../../components/ConfirmModal';
import Icon from '../../components/Icon';
import * as Expensicons from '../../components/Icon/Expensicons';
import ScreenWrapper from '../../components/ScreenWrapper';
Expand Down Expand Up @@ -37,7 +38,7 @@ class WorkspaceInitialPage extends React.Component {
this.confirmDeleteAndHideModal = this.confirmDeleteAndHideModal.bind(this);

this.state = {
isDeleteModalOpen: false,
// isDeleteModalOpen: false,
};
}

Expand All @@ -52,8 +53,9 @@ class WorkspaceInitialPage extends React.Component {
* Toggle delete confirm modal visibility
* @param {Boolean} shouldOpen
*/
// eslint-disable-next-line
toggleDeleteModal(shouldOpen) {
this.setState({isDeleteModalOpen: shouldOpen});
// this.setState({isDeleteModalOpen: shouldOpen});
}

/**
Expand Down Expand Up @@ -131,7 +133,8 @@ class WorkspaceInitialPage extends React.Component {
}, {
icon: Expensicons.Trashcan,
text: this.props.translate('workspace.common.delete'),
onSelected: () => this.setState({isDeleteModalOpen: true}),

// onSelected: () => this.setState({isDeleteModalOpen: true}),
},
]}
threeDotsAnchorPosition={styles.threeDotsPopoverOffset}
Expand Down Expand Up @@ -204,6 +207,7 @@ class WorkspaceInitialPage extends React.Component {
))}
</View>
</ScrollView>
{/* TODO: this is to test something in staging we are having a hard time figuring out in dev. Will revert once we test on staging - More details in https://github.com/Expensify/App/issues/8791
<ConfirmModal
title={this.props.translate('workspace.common.delete')}
isVisible={this.state.isDeleteModalOpen}
Expand All @@ -213,7 +217,8 @@ class WorkspaceInitialPage extends React.Component {
confirmText={this.props.translate('common.delete')}
cancelText={this.props.translate('common.cancel')}
danger
/>
>
*/}
</ScreenWrapper>
);
}
Expand Down

0 comments on commit c132c97

Please sign in to comment.