Skip to content

Commit efebcdb

Browse files
Merge pull request #4520 from OneCommunityGlobal/sundar/fix-4150
Sundar: 🔥 the props issue in WeeklySummariesToggleFilter.jsx
2 parents 4b04a21 + 838a84d commit efebcdb

1 file changed

Lines changed: 12 additions & 1 deletion

File tree

src/components/WeeklySummariesReport/WeeklySummariesToggleFilter.jsx

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
import PropTypes from 'prop-types';
12
import { Label } from 'reactstrap';
23
import styles from './WeeklySummariesReport.module.css';
34
import ReactTooltip from 'react-tooltip';
@@ -9,6 +10,7 @@ export default function WeeklySummariesToggleFilter({
910
hasPermissionToFilter,
1011
editable,
1112
formId,
13+
hasPermission,
1214
}) {
1315
const handleTrophyToggleChange = () => {
1416
toggleField(setState, 'selectedTrophies');
@@ -64,7 +66,7 @@ export default function WeeklySummariesToggleFilter({
6466
</div>
6567
</div>
6668
)}
67-
{(hasPermissionToFilter || props.hasPermission('highlightEligibleBios')) && (
69+
{(hasPermissionToFilter || hasPermission?.('highlightEligibleBios')) && (
6870
<div className={`${styles.filterStyle} ml-3`} style={{ minWidth: 'max-content' }}>
6971
<span>Filter by Bio Status</span>
7072
<div className={styles.switchToggleControl}>
@@ -129,3 +131,12 @@ export default function WeeklySummariesToggleFilter({
129131
</div>
130132
);
131133
}
134+
135+
WeeklySummariesToggleFilter.propTypes = {
136+
state: PropTypes.object.isRequired,
137+
setState: PropTypes.func.isRequired,
138+
hasPermissionToFilter: PropTypes.bool,
139+
editable: PropTypes.bool,
140+
formId: PropTypes.string.isRequired,
141+
hasPermission: PropTypes.func,
142+
};

0 commit comments

Comments
 (0)