Skip to content
This repository has been archived by the owner on Sep 14, 2021. It is now read-only.

fix(Header): Fix employer site link #211

Merged
merged 3 commits into from
May 30, 2017
Merged

Conversation

markdalgleish
Copy link
Member

No description provided.

@@ -1,3 +1,3 @@
export default locale => locale.toLowerCase() === 'NZ' ?
export default locale => locale.toLowerCase() === 'nz' ?
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would prefer to regex this /^nz$/i.test(locale)

Copy link
Contributor

@michaeltaranto michaeltaranto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎨

pokle
pokle previously approved these changes May 30, 2017
Copy link

@pokle pokle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@michaeltaranto michaeltaranto dismissed pokle’s stale review May 30, 2017 05:21

Re-review please

Copy link

@pokle pokle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Again

@markdalgleish markdalgleish merged commit 35bf7d1 into master May 30, 2017
@markdalgleish markdalgleish deleted the fix-employer-site-link branch May 30, 2017 05:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants