Skip to content

Commit

Permalink
Merge pull request #888 from ssciolla/issue-644-remove-get-derived
Browse files Browse the repository at this point in the history
Remove `ContentPage.getDerviedStateFromProps` to fix navigation bug (#644)
  • Loading branch information
dmols authored Jul 24, 2024
2 parents 80a6c7f + 9107666 commit 0839145
Showing 1 changed file with 0 additions and 11 deletions.
11 changes: 0 additions & 11 deletions assets/js/Components/ContentPage.js
Original file line number Diff line number Diff line change
Expand Up @@ -77,17 +77,6 @@ class ContentPage extends React.Component {
}
}

static getDerivedStateFromProps(props, state) {
const stateActiveIssue = state.activeIssue
const propsActiveIssue = stateActiveIssue && props.report.issues[stateActiveIssue.id]
if(propsActiveIssue && propsActiveIssue.status !== stateActiveIssue.status) {
return {
activeIssue: propsActiveIssue
}
}
return null
}

handleSearchTerm = (e, val) => {
this.setState({searchTerm: val, filteredIssues: [], tableSettings: Object.assign({}, this.state.tableSettings, {pageNum: 0})});
}
Expand Down

0 comments on commit 0839145

Please sign in to comment.