Skip to content

Commit 60dee93

Browse files
committed
fix initial files proptype, linting for story
1 parent 8192f73 commit 60dee93

File tree

4 files changed

+54
-36
lines changed

4 files changed

+54
-36
lines changed

src/components/FilesTable/FilesTable.jsx

Lines changed: 15 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -57,19 +57,21 @@ const FilesTable = ({ addClass, files }) => {
5757

5858
FilesTable.propTypes = {
5959
addClass: PropTypes.string,
60-
files: PropTypes.shape([{
61-
contentLength: PropTypes.string.isRequired,
62-
content_length: PropTypes.number.isRequired,
63-
content_type: PropTypes.string.isRequired,
64-
createdAt: PropTypes.string.isRequired,
65-
created_at: PropTypes.string.isRequired,
66-
download: PropTypes.string.isRequired,
67-
filename: PropTypes.string.isRequired,
68-
href: PropTypes.string.isRequired,
69-
status: PropTypes.string.isRequired,
70-
uploadedBy: PropTypes.string.isRequired,
71-
uuid: PropTypes.string.isRequired,
72-
}]).isRequired,
60+
files: PropTypes.arrayOf(
61+
PropTypes.shape({
62+
contentLength: PropTypes.string.isRequired,
63+
content_length: PropTypes.number.isRequired,
64+
content_type: PropTypes.string.isRequired,
65+
createdAt: PropTypes.string.isRequired,
66+
created_at: PropTypes.string.isRequired,
67+
download: PropTypes.string.isRequired,
68+
filename: PropTypes.string.isRequired,
69+
href: PropTypes.string.isRequired,
70+
status: PropTypes.string.isRequired,
71+
uploadedBy: PropTypes.string.isRequired,
72+
uuid: PropTypes.string.isRequired,
73+
}),
74+
).isRequired,
7375
// TODO(summer-cook): add this back in. see comment above.
7476
// handleDeleteFile: PropTypes.func.isRequired,
7577
}

src/compounds/ActionsGroup/ActionsGroup.jsx

Lines changed: 15 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -62,19 +62,21 @@ const ActionsGroup = ({ initialFiles }) => {
6262

6363
ActionsGroup.propTypes = {
6464
// handleSendingMessagesOrFiles: PropTypes.func.isRequired,
65-
initialFiles: PropTypes.shape([{
66-
contentLength: PropTypes.string.isRequired,
67-
content_length: PropTypes.number.isRequired,
68-
content_type: PropTypes.string.isRequired,
69-
createdAt: PropTypes.string.isRequired,
70-
created_at: PropTypes.string.isRequired,
71-
download: PropTypes.string.isRequired,
72-
filename: PropTypes.string.isRequired,
73-
href: PropTypes.string.isRequired,
74-
status: PropTypes.string.isRequired,
75-
uploadedBy: PropTypes.string.isRequired,
76-
uuid: PropTypes.string.isRequired,
77-
}]).isRequired,
65+
initialFiles: PropTypes.arrayOf(
66+
PropTypes.shape({
67+
contentLength: PropTypes.string.isRequired,
68+
content_length: PropTypes.number.isRequired,
69+
content_type: PropTypes.string.isRequired,
70+
createdAt: PropTypes.string.isRequired,
71+
created_at: PropTypes.string.isRequired,
72+
download: PropTypes.string.isRequired,
73+
filename: PropTypes.string.isRequired,
74+
href: PropTypes.string.isRequired,
75+
status: PropTypes.string.isRequired,
76+
uploadedBy: PropTypes.string.isRequired,
77+
uuid: PropTypes.string.isRequired,
78+
}),
79+
).isRequired,
7880
}
7981

8082
export default ActionsGroup

src/compounds/ActionsGroup/ActionsGroup.stories.jsx

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -16,16 +16,16 @@ Default.args = {
1616
console.log(`attached file(s): ${files.map((file) => file.split('name=')[1])}`)
1717
},
1818
initialFiles: [{
19-
contentLength: "938.57 KB",
19+
contentLength: '938.57 KB',
2020
content_length: 961096,
21-
content_type: "image/png",
22-
createdAt: " Dec 22 2022 at 12:56:48 PM",
23-
created_at: "2022-12-22T10:56:48.405-08:00",
24-
download: "/quote_groups/10bd825f-c7e0-4c07-bd9e-f7982b21f435/attachments/nepal5.png",
25-
filename: "nepal5.png",
26-
href: "https://acme.scientist.com/secure_attachments/4611434c-ba99-481e-84ee-37e1fdbd203e",
21+
content_type: 'image/png',
22+
createdAt: ' Dec 22 2022 at 12:56:48 PM',
23+
created_at: '2022-12-22T10:56:48.405-08:00',
24+
download: '/quote_groups/10bd825f-c7e0-4c07-bd9e-f7982b21f435/attachments/nepal5.png',
25+
filename: 'nepal5.png',
26+
href: 'https://acme.scientist.com/secure_attachments/4611434c-ba99-481e-84ee-37e1fdbd203e',
2727
status: null,
28-
uploadedBy: "Summer Cook",
29-
uuid: "4611434c-ba99-481e-84ee-37e1fdbd203e",
28+
uploadedBy: 'Summer Cook',
29+
uuid: '4611434c-ba99-481e-84ee-37e1fdbd203e',
3030
}],
3131
}

src/compounds/ActionsGroup/actions/ViewFiles.jsx

Lines changed: 15 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -133,7 +133,21 @@ const ViewFiles = ({ initialFiles, handleClose, show }) => {
133133
}
134134

135135
ViewFiles.propTypes = {
136-
initialFiles: PropTypes.shape([{}]).isRequired,
136+
initialFiles: PropTypes.arrayOf(
137+
PropTypes.shape({
138+
contentLength: PropTypes.string.isRequired,
139+
content_length: PropTypes.number.isRequired,
140+
content_type: PropTypes.string.isRequired,
141+
createdAt: PropTypes.string.isRequired,
142+
created_at: PropTypes.string.isRequired,
143+
download: PropTypes.string.isRequired,
144+
filename: PropTypes.string.isRequired,
145+
href: PropTypes.string.isRequired,
146+
status: PropTypes.string.isRequired,
147+
uploadedBy: PropTypes.string.isRequired,
148+
uuid: PropTypes.string.isRequired,
149+
}),
150+
).isRequired,
137151
handleClose: PropTypes.func.isRequired,
138152
show: PropTypes.bool.isRequired,
139153
}

0 commit comments

Comments
 (0)