fedora-qa / landingpage

#81 changed class based components to functional components
Merged by lbrabec. Opened by manishakanyal.
fedora-qa/ manishakanyal/landingpage functional-based-components  into  master

Download 81.patch

@lbrabec I have changed all components to functional, please have a look!

rebased onto a38da496d1426554e355bfc3610a41b13fbfc412

Is there a reason why this is function App() instead of const App = () =>? If not, please change it to latter so it is consistent with other components.

Other than my previous comment, LGTM.

rebased onto 29f47225448e5c8a92db051232502a445017e032

Is there a reason why this is function App() instead of const App = () =>? If not, please change it to latter so it is consistent with other components.

@lbrabec, I have changed this to an arrow function, please have a look!

Great! Merging.

Pull-Request has been merged by lbrabec
Metadata
None
No Tags
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.