Skip to content
This repository has been archived by the owner on Mar 21, 2019. It is now read-only.

Refactor header #52

Open
wants to merge 13 commits into
base: master
Choose a base branch
from
Open

Refactor header #52

wants to merge 13 commits into from

Conversation

itsmetambui
Copy link

What does this PR do?

  • Extract common Header code to a single component, related to this issue.
  • Extract logic to load production.min.js script every time the route change to App.js.
  • Add now.json for Now deployment.

Where should the review start?

  • Live demo here.
  • Header component in src/components/global/Header.js

Motivation

  • This is my first PR, feels good :D, please help to take a look.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
1 participant