Project

General

Profile

Feature #27853

Integrate modals with Redux store

Added by Jeremy Lenz 11 months ago. Updated 8 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
JavaScript stack
Target version:
-
Difficulty:
Triaged:
No
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

Depends on https://github.com/theforeman/foreman/pull/7039
User stories:
Users should be able to check a status of a modal by id
Users should be able to close/open Modals by id

Associated revisions

Revision 9c18affc (diff)
Added by Jeremy Lenz 8 months ago

Fixes #27853: Connect ForemanModal to Redux store

For unsquashed commits, see jeremylenz/27853-modals-redux-unsquashed

Connect ForemanModal React component to Redux store.
Add useForemanModal hook
for reusing stateful logic in any ForemanModal component.
Modal state is now controlled by id with Redux actions.
Uses react-redux v7.1,
which includes useSelector hook
and allows use of Redux without connect,
mapStateToProps or mapDispatchToProps.

History

#1 Updated by Jeremy Lenz 11 months ago

  • Tracker changed from Bug to Feature

#2 Updated by Jeremy Lenz 11 months ago

  • Category set to JavaScript stack

#3 Updated by The Foreman Bot 10 months ago

  • Pull request https://github.com/theforeman/foreman/pull/7105 added

#4 Updated by The Foreman Bot 9 months ago

  • Status changed from New to Ready For Testing

#5 Updated by The Foreman Bot 9 months ago

  • Assignee set to Jeremy Lenz

#6 Updated by The Foreman Bot 8 months ago

  • Pull request https://github.com/theforeman/foreman/pull/7265 added

#7 Updated by Jeremy Lenz 8 months ago

  • Pull request deleted (https://github.com/theforeman/foreman/pull/7265)

#8 Updated by The Foreman Bot 8 months ago

  • Fixed in Releases 2.0.0 added

#9 Updated by Jeremy Lenz 8 months ago

  • Status changed from Ready For Testing to Closed

Also available in: Atom PDF