Skip to content

Comments

Refactor AerodromeDropdownContainer to modern way of implementing it#605

Merged
rzueger merged 1 commit intodevelopfrom
pr/refactor-ad-dropdown-container
Feb 19, 2026
Merged

Refactor AerodromeDropdownContainer to modern way of implementing it#605
rzueger merged 1 commit intodevelopfrom
pr/refactor-ad-dropdown-container

Conversation

@rzueger
Copy link
Member

@rzueger rzueger commented Feb 19, 2026

  • Functional component with destructured props instead of a class
  • useSelector replaces mapStateToProps — reads directly from the Redux store
  • useDispatch replaces mapDispatchToProps — you no longer need connect() at all
  • useEffect with [] replaces componentWillMount — runs once after first render. (componentWillMount actually ran before render, but for data fetching the difference is negligible and useEffect is the correct modern pattern)
  • PropTypes for injected Redux props are removed since they're no longer passed from a parent — only the truly external props remain

- Functional component with destructured props instead of a class
- useSelector replaces mapStateToProps — reads directly from the Redux
  store
- useDispatch replaces mapDispatchToProps — you no longer need connect()
  at all
- useEffect with [] replaces componentWillMount — runs once after first
  render. (componentWillMount actually ran before render, but for data
  fetching the difference is negligible and useEffect is the correct
  modern pattern)
- PropTypes for injected Redux props are removed since they're no longer
  passed from a parent — only the truly external props remain
@rzueger rzueger merged commit a6e1a10 into develop Feb 19, 2026
2 checks passed
@rzueger rzueger deleted the pr/refactor-ad-dropdown-container branch February 19, 2026 14:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant