Skip to content

Commit

Permalink
Merge pull request #26924 from daordonez11/fix/avoid-task-pages
Browse files Browse the repository at this point in the history
  • Loading branch information
francoisl authored Sep 11, 2023
2 parents f4affe5 + d45118b commit b2f3d44
Show file tree
Hide file tree
Showing 3 changed files with 22 additions and 1 deletion.
8 changes: 7 additions & 1 deletion src/pages/tasks/TaskAssigneeSelectorModal.js
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ import compose from '../../libs/compose';
import personalDetailsPropType from '../personalDetailsPropType';
import reportPropTypes from '../reportPropTypes';
import ROUTES from '../../ROUTES';

import * as ReportUtils from '../../libs/ReportUtils';
import * as Task from '../../libs/actions/Task';

const propTypes = {
Expand Down Expand Up @@ -124,6 +124,12 @@ function TaskAssigneeSelectorModal(props) {
return props.reports[`${ONYXKEYS.COLLECTION.REPORT}${props.route.params.reportID}`];
}, [props.reports, props.route.params]);

if (report && !ReportUtils.isTaskReport(report)) {
Navigation.isNavigationReady().then(() => {
Navigation.dismissModal(report.reportID);
});
}

const sections = useMemo(() => {
const sectionsList = [];
let indexOffset = 0;
Expand Down
7 changes: 7 additions & 0 deletions src/pages/tasks/TaskDescriptionPage.js
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,8 @@ import * as Task from '../../libs/actions/Task';
import CONST from '../../CONST';
import focusAndUpdateMultilineInputRange from '../../libs/focusAndUpdateMultilineInputRange';
import * as Browser from '../../libs/Browser';
import * as ReportUtils from '../../libs/ReportUtils';
import Navigation from '../../libs/Navigation/Navigation';

const propTypes = {
/** Current user session */
Expand Down Expand Up @@ -46,6 +48,11 @@ function TaskDescriptionPage(props) {
[props],
);

if (!ReportUtils.isTaskReport(props.report)) {
Navigation.isNavigationReady().then(() => {
Navigation.dismissModal(props.report.reportID);
});
}
const inputRef = useRef(null);

return (
Expand Down
8 changes: 8 additions & 0 deletions src/pages/tasks/TaskTitlePage.js
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,8 @@ import reportPropTypes from '../reportPropTypes';
import compose from '../../libs/compose';
import * as Task from '../../libs/actions/Task';
import CONST from '../../CONST';
import * as ReportUtils from '../../libs/ReportUtils';
import Navigation from '../../libs/Navigation/Navigation';

const propTypes = {
/** The report currently being looked at */
Expand Down Expand Up @@ -58,6 +60,12 @@ function TaskTitlePage(props) {
[props],
);

if (!ReportUtils.isTaskReport(props.report)) {
Navigation.isNavigationReady().then(() => {
Navigation.dismissModal(props.report.reportID);
});
}

const inputRef = useRef(null);

return (
Expand Down

0 comments on commit b2f3d44

Please sign in to comment.