Skip to content

Commit

Permalink
Merge pull request #4122 from alisman/noExpMessage
Browse files Browse the repository at this point in the history
Retire expression tab user message
  • Loading branch information
alisman authored Jan 12, 2022
2 parents 41d0a5a + 1b2ddb8 commit 7fe967c
Showing 1 changed file with 6 additions and 40 deletions.
46 changes: 6 additions & 40 deletions src/pages/resultsView/ResultsViewPage.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,9 @@ import OQLTextArea, {
} from 'shared/components/GeneSelectionBox/OQLTextArea';
import browser from 'bowser';
import { QueryStore } from '../../shared/components/query/QueryStore';
import UserMessager from 'shared/components/userMessager/UserMessage';
import UserMessager, {
IUserMessage,
} from 'shared/components/userMessager/UserMessage';
import { HelpWidget } from 'shared/components/HelpWidget/HelpWidget';

export function initStore(
Expand Down Expand Up @@ -545,50 +547,14 @@ export default class ResultsViewPage extends React.Component<
});
}

readonly userMessages = remoteData({
readonly userMessages = remoteData<IUserMessage[]>({
await: () => [
this.resultsViewPageStore.expressionProfiles,
this.resultsViewPageStore.studies,
],
invoke: () => {
// TODO: This is only here temporarily to shepherd users from
// now-deleted Expression tab to the Plots tab.
// Remove a few months after 10/2020
if (
this.resultsViewPageStore.expressionProfiles.result.length >
0 &&
this.resultsViewPageStore.studies.result.length > 1
) {
return Promise.resolve([
{
dateEnd: 10000000000000000000,
content: (
<span>
Looking for the <strong>Expression</strong> tab?
{` `}
That functionality is now available{` `}
<a
style={{
color: 'white',
textDecoration: 'underline',
}}
onClick={() =>
this.urlWrapper.updateURL(
{},
`results/${ResultsViewTab.EXPRESSION_REDIRECT}`
)
}
>
in the <strong>Plots</strong> tab.
</a>
</span>
),
id: '2020_merge_expression_to_plots',
},
]);
} else {
return Promise.resolve([]);
}
// TODO: make this configurable from outside app
return Promise.resolve([]);
},
});

Expand Down

0 comments on commit 7fe967c

Please sign in to comment.