Skip to content

Commit 536f234

Browse files
authored
Merge pull request #2545 from bcgov/fix/kevin-2536
fix: director assessment statement
2 parents 28416cb + 3aa7295 commit 536f234

File tree

1 file changed

+24
-23
lines changed

1 file changed

+24
-23
lines changed

frontend/src/views/ComplianceReports/components/HistoryCard.jsx

Lines changed: 24 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,17 @@
1-
import React, { useMemo } from 'react'
1+
import BCTypography from '@/components/BCTypography/index.jsx'
2+
import { StyledListItem } from '@/components/StyledListItem.jsx'
3+
import { COMPLIANCE_REPORT_STATUSES } from '@/constants/statuses.js'
4+
import { useCurrentUser } from '@/hooks/useCurrentUser.js'
5+
import { timezoneFormatter } from '@/utils/formatters.js'
6+
import { ExpandMore } from '@mui/icons-material'
27
import { List, ListItemText, styled } from '@mui/material'
38
import MuiAccordion from '@mui/material/Accordion'
9+
import MuiAccordionDetails from '@mui/material/AccordionDetails'
410
import MuiAccordionSummary, {
511
accordionSummaryClasses
612
} from '@mui/material/AccordionSummary'
7-
import MuiAccordionDetails from '@mui/material/AccordionDetails'
8-
import { useCurrentUser } from '@/hooks/useCurrentUser.js'
13+
import { useMemo } from 'react'
914
import { useTranslation } from 'react-i18next'
10-
import { COMPLIANCE_REPORT_STATUSES } from '@/constants/statuses.js'
11-
import BCTypography from '@/components/BCTypography/index.jsx'
12-
import { StyledListItem } from '@/components/StyledListItem.jsx'
13-
import { timezoneFormatter } from '@/utils/formatters.js'
14-
import { ExpandMore } from '@mui/icons-material'
1515

1616
const Accordion = styled((props) => (
1717
<MuiAccordion disableGutters elevation={0} square {...props} />
@@ -85,6 +85,22 @@ export const HistoryCard = ({ report, defaultExpanded = false }) => {
8585
{filteredHistory.length > 0 && (
8686
<AccordionDetails>
8787
<List>
88+
{report.assessmentStatement &&
89+
((!isGovernmentUser &&
90+
report.currentStatus.status === 'Assessed') ||
91+
isGovernmentUser) && (
92+
<StyledListItem disablePadding>
93+
<ListItemText
94+
data-test="list-item"
95+
primaryTypographyProps={{ variant: 'body4' }}
96+
>
97+
<strong>
98+
{t('report:complianceReportHistory.directorStatement')}:
99+
</strong>{' '}
100+
{report.assessmentStatement}
101+
</ListItemText>
102+
</StyledListItem>
103+
)}
88104
{filteredHistory.map((item, index) => (
89105
<StyledListItem key={index} disablePadding>
90106
<ListItemText
@@ -146,21 +162,6 @@ export const HistoryCard = ({ report, defaultExpanded = false }) => {
146162
})}
147163
</ListItemText>
148164
</StyledListItem>
149-
{report.assessmentStatement && (
150-
<StyledListItem disablePadding>
151-
<ListItemText
152-
primaryTypographyProps={{ variant: 'body4' }}
153-
>
154-
<strong>
155-
{t(
156-
'report:complianceReportHistory.directorStatement'
157-
)}
158-
:&nbsp;
159-
</strong>
160-
{report.assessmentStatement}
161-
</ListItemText>
162-
</StyledListItem>
163-
)}
164165
</List>
165166
)}
166167
</StyledListItem>

0 commit comments

Comments
 (0)