Skip to content

Commit ce2229c

Browse files
committed
status is allowed to be null- fix proptype
1 parent c8ce839 commit ce2229c

File tree

3 files changed

+19
-4
lines changed

3 files changed

+19
-4
lines changed

src/components/FilesTable/FilesTable.jsx

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@ FilesTable.propTypes = {
6464
download: PropTypes.string.isRequired,
6565
fileName: PropTypes.string.isRequired,
6666
href: PropTypes.string.isRequired,
67-
status: PropTypes.string.isRequired,
67+
status: PropTypes.string,
6868
uploadedBy: PropTypes.string.isRequired,
6969
uuid: PropTypes.string.isRequired,
7070
}),
@@ -75,6 +75,9 @@ FilesTable.propTypes = {
7575

7676
FilesTable.defaultProps = {
7777
addClass: '',
78+
files: [{
79+
status: null
80+
}]
7881
}
7982

8083
export default FilesTable

src/compounds/ActionsGroup/ActionsGroup.jsx

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -69,11 +69,17 @@ ActionsGroup.propTypes = {
6969
download: PropTypes.string.isRequired,
7070
fileName: PropTypes.string.isRequired,
7171
href: PropTypes.string.isRequired,
72-
status: PropTypes.string.isRequired,
72+
status: PropTypes.string,
7373
uploadedBy: PropTypes.string.isRequired,
7474
uuid: PropTypes.string.isRequired,
7575
}),
7676
).isRequired,
7777
}
7878

79+
ActionsGroup.defaultProps = {
80+
initialFiles: [{
81+
status: null
82+
}]
83+
}
84+
7985
export default ActionsGroup

src/compounds/ActionsGroup/actions/ViewFiles.jsx

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,7 @@ const ViewFiles = ({ initialFiles, handleClose, show }) => {
100100
)
101101
})}
102102
</ListGroup> */}
103-
<Tabs defaultActiveKey='files' id='document-tabs'>
103+
<Tabs defaultActiveKey='files' id='document-tabs' justify fill>
104104
{documentTabs && documentTabs.map((tab) => {
105105
const { eventKey, title, status } = tab
106106
const filteredFiles = initialFiles.filter((f) => (status === f.status) || (status === 'Other File' && f.status === null))
@@ -135,7 +135,7 @@ ViewFiles.propTypes = {
135135
download: PropTypes.string.isRequired,
136136
fileName: PropTypes.string.isRequired,
137137
href: PropTypes.string.isRequired,
138-
status: PropTypes.string.isRequired,
138+
status: PropTypes.string,
139139
uploadedBy: PropTypes.string.isRequired,
140140
uuid: PropTypes.string.isRequired,
141141
}),
@@ -144,4 +144,10 @@ ViewFiles.propTypes = {
144144
show: PropTypes.bool.isRequired,
145145
}
146146

147+
ViewFiles.defaultProps = {
148+
initialFiles: [{
149+
status: null
150+
}]
151+
}
152+
147153
export default ViewFiles

0 commit comments

Comments
 (0)