Skip to content

Commit 89d6c4b

Browse files
committed
Added prop type validations for Facets
1 parent f6b84da commit 89d6c4b

File tree

3 files changed

+13
-2
lines changed

3 files changed

+13
-2
lines changed

src/components/Facet.js

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
import PropTypes from "prop-types";
12
import React from "react";
23

34
import "./Facet.css";
@@ -35,4 +36,12 @@ function Facet({ name, onRemove, onSelect, options, value }) {
3536
);
3637
}
3738

39+
Facet.propTypes = {
40+
name: PropTypes.string.isRequired,
41+
onRemove: PropTypes.func.isRequired,
42+
onSelect: PropTypes.func.isRequired,
43+
options: PropTypes.arrayOf(PropTypes.object).isRequired,
44+
value: PropTypes.string
45+
};
46+
3847
export default Facet;

src/containers/Facets.js

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,9 @@ function findFacetValueInFilters(name, filters) {
1818
class FacetsContainer extends Component {
1919
static propTypes = {
2020
addFilter: PropTypes.func.isRequired,
21-
facets: PropTypes.object.isRequired
21+
filters: PropTypes.arrayOf(PropTypes.object).isRequired,
22+
facets: PropTypes.object.isRequired,
23+
removeFilter: PropTypes.func.isRequired
2224
};
2325

2426
render() {

src/containers/Results.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ function formatResultFields(result) {
3737
}
3838
class ResultsContainer extends Component {
3939
static propTypes = {
40-
results: PropTypes.array.isRequired
40+
results: PropTypes.arrayOf(PropTypes.object).isRequired
4141
};
4242

4343
render() {

0 commit comments

Comments
 (0)