Skip to content

Commit f73ce35

Browse files
authored
fix mskcc study usage acknowledgement (#5023)
1 parent 15011e0 commit f73ce35

File tree

2 files changed

+10
-27
lines changed

2 files changed

+10
-27
lines changed

src/appShell/App/usageAgreements/StudyAgreement.tsx

Lines changed: 7 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -28,11 +28,12 @@ export const StudyAgreement: React.FunctionComponent<{}> = function({}) {
2828
<UsageAgreement
2929
alertMessage={
3030
<>
31-
<span style={{ color: 'red' }}>WARNING:</span>
32-
&nbsp;All URLs in this website are private - do NOT include
33-
in manuscripts.
31+
<span style={{ color: 'red' }}>Attention:</span>
32+
&nbsp;Please read and follow the rules about usage of MSK
33+
clinical sequencing data in manuscripts.
3434
</>
3535
}
36+
dismissButtonText={'Acknowledge'}
3637
persistenceKey={STUDY_VIEW_WARNING_PERSISTENCE_KEY}
3738
expirationInDays={90}
3839
clauses={[
@@ -55,32 +56,14 @@ export const StudyAgreement: React.FunctionComponent<{}> = function({}) {
5556
with any questions about getting the data transferred to the
5657
public cBioPortal.
5758
</>,
58-
<>
59-
When adding a link to a cBioPortal cohort in a manuscript,{' '}
60-
<strong>
61-
I will not link to this private portal (
62-
{window.location.hostname})
63-
</strong>
64-
, but will instead link to this study on the public
65-
cBioPortal (
66-
<a href="https://www.cbioportal.org/" target="_blank">
67-
cbioportal.org
68-
</a>
69-
). Contact{' '}
70-
<a href="mailto:[email protected]">
71-
72-
</a>{' '}
73-
with any questions about getting the data transferred to the
74-
public cBioPortal.
75-
</>,
7659
<>
7760
I have read the{' '}
7861
<a
79-
href="https://cmo.mskcc.org/index.php/msk-impact/"
62+
href="https://mskcc.sharepoint.com/sites/pub-ResearchDG/SitePages/Home.aspx?ga=1"
8063
target="_blank"
8164
>
82-
MSK-IMPACT Data publication guidelines (intranet/VPN
83-
only)
65+
I have read the MSK-IMPACT Research Data Governance
66+
publication guidelines.
8467
</a>
8568
.
8669
</>,

src/shared/components/UsageAgreement.tsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@ interface IUsageAgreement {
1515
clauses: JSX.Element[];
1616
useCheckboxes?: boolean;
1717
expirationInDays?: number;
18+
dismissButtonText?: string;
1819
}
1920

2021
@observer
@@ -68,7 +69,6 @@ export default class UsageAgreement extends React.Component<
6869
'true',
6970
this.expirationInSeconds
7071
);
71-
//localStorage.setItem(this.props.persistenceKey, 'true');
7272
}
7373

7474
@autobind
@@ -111,7 +111,7 @@ export default class UsageAgreement extends React.Component<
111111
bsSize="xsmall"
112112
onClick={this.handleModalShow}
113113
>
114-
Dismiss
114+
{this.props.dismissButtonText || 'Dismiss'}
115115
</Button>
116116
</ButtonGroup>
117117
</Alert>
@@ -122,7 +122,7 @@ export default class UsageAgreement extends React.Component<
122122
container={this}
123123
aria-labelledby="contained-modal-title"
124124
>
125-
<Modal.Header closeButton>
125+
<Modal.Header>
126126
<Modal.Title
127127
id="contained-modal-title"
128128
style={{ textAlign: 'center' }}

0 commit comments

Comments
 (0)