Project

General

Profile

Actions
Copy link

Bug #26071

open

Refactor Higher Order Components for Check Organization component (React)

Added by John Mitsch about 6 years ago. Updated 8 months ago.

Status:
New
Priority:
Normal
Assignee:
John Mitsch
Category:
Organizations and Locations
Target version:
-
Difficulty:
Triaged:
Yes
Bugzilla link:
Pull request:
Fixed in Releases:
Found in Releases:
Red Hat JIRA:

Description

This comes from this discussion https://github.com/Katello/katello/pull/7954#discussion_r256451637

We would like to keep the CheckOrg component snapshot 'disconnected' from Redux to make smaller and more manageable snapshots.

This could be done with a HOC, which is suggested in the discussion. I started with an implementation for this but was blocked by some issues, so this is a follow up to investigate this more (and not block the PR)

There may be additional improvements in this area around the HOCs. Redux has a compose method that could be used to 'stack' HOCs on a component https://redux.js.org/api/compose

My try is in https://github.com/johnpmitsch/katello/commit/5b746864d65f764e1b8b25c7dcf9f68817b5ae10

Actions
Copy link

Also available in: Atom PDF