fedora-qa / landingpage

#65 Added mapStateToProps to landingpage sub-componets(Blockers.js, Events.js, Minutes.js, Timeline.js) for better access of state
Closed by lbrabec. Opened by kunalprakash1309.
fedora-qa/ kunalprakash1309/landingpage basic-redux  into  master

Download 65.patch

despite of having state store which contain all the state. Why we
are passing state as a props to component like Timeline.js, Event.js etc.

By using connect(https://react-redux.js.org/api/connect) for accessing particular state required by respective components despite of passing the state as props to respective components.

1 new commit added

  • same method i.e mapStateToProps added to fraction of wizard section

Thanks.

Please squash the two commits to one.

rebased onto fb181ca8d93142d507979cca22380559a33bb9bf
kunalprakash1309 commented

@lbrabec Squashing of commits are done.

Metadata Update from @lbrabec:
- Pull-request tagged with: review done

Thanks for this pull request, however, we chose functional components with hooks, closing this PR.

Pull-Request has been closed by lbrabec
Metadata
None

review done

Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.