Skip to content

Commit 294b297

Browse files
committed
Fix linting and prop type errors
1 parent 954d632 commit 294b297

File tree

4 files changed

+15
-11
lines changed

4 files changed

+15
-11
lines changed

client/modules/IDE/components/CollectionList/CollectionListRow.jsx

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -227,6 +227,13 @@ CollectionListRowBase.propTypes = {
227227
owner: PropTypes.shape({
228228
username: PropTypes.string.isRequired,
229229
}).isRequired,
230+
createdAt: PropTypes.string.isRequired,
231+
updatedAt: PropTypes.string.isRequired,
232+
items: PropTypes.arrayOf(PropTypes.shape({
233+
project: PropTypes.shape({
234+
id: PropTypes.string.isRequired
235+
})
236+
}))
230237
}).isRequired,
231238
project: PropTypes.shape({
232239
id: PropTypes.string.isRequired,

client/modules/IDE/components/SketchList.jsx

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -437,15 +437,6 @@ SketchList.propTypes = {
437437
createdAt: PropTypes.string.isRequired,
438438
updatedAt: PropTypes.string.isRequired
439439
})).isRequired,
440-
collection: PropTypes.shape({
441-
id: PropTypes.string.isRequired,
442-
name: PropTypes.string.isRequired,
443-
items: PropTypes.arrayOf(PropTypes.shape({
444-
project: PropTypes.shape({
445-
id: PropTypes.string.isRequired,
446-
}),
447-
})),
448-
}).isRequired,
449440
username: PropTypes.string,
450441
loading: PropTypes.bool.isRequired,
451442
toggleDirectionForField: PropTypes.func.isRequired,

client/modules/User/components/Collection.jsx

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -89,9 +89,13 @@ CollectionItemRowBase.propTypes = {
8989
name: PropTypes.string.isRequired
9090
}).isRequired,
9191
item: PropTypes.shape({
92+
createdAt: PropTypes.string.isRequired,
9293
project: PropTypes.shape({
9394
id: PropTypes.string.isRequired,
9495
name: PropTypes.string.isRequired,
96+
user: PropTypes.shape({
97+
username: PropTypes.string.isRequired
98+
})
9599
}).isRequired,
96100
}).isRequired,
97101
user: PropTypes.shape({
@@ -204,7 +208,9 @@ class Collection extends React.Component {
204208
<div className="collection-metadata__column--left">
205209
<h2 className="collection-metadata__name">
206210
{
207-
this.isOwner() ? <EditableInput value={name} onChange={handleEditCollectionName} validate={value => value !== ''} /> : name
211+
this.isOwner() ?
212+
<EditableInput value={name} onChange={handleEditCollectionName} validate={value => value !== ''} /> :
213+
name
208214
}
209215
</h2>
210216

client/modules/User/components/DashboardTabSwitcher.jsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ const Tab = ({ children, isSelected, to }) => {
2323
};
2424

2525
Tab.propTypes = {
26-
children: PropTypes.element.isRequired,
26+
children: PropTypes.string.isRequired,
2727
isSelected: PropTypes.bool.isRequired,
2828
to: PropTypes.string.isRequired,
2929
};

0 commit comments

Comments
 (0)