fix(components): pass href to NavItem link

[connor-baer]

Follow up to #466.

Purpose

In the original PR, I forgot to pass the href through to the Link component in the NavItem.

Approach and changes

• pass href to NavItem link
• remove link underline

Definition of done

• Development completed
• Reviewers assigned
• Unit and integration tests
• Meets minimum browser support
• Meets accessibility requirements

[codecov]

Codecov Report

Merging #492 into canary will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           canary     #492      +/-   ##
==========================================
+ Coverage   79.57%   79.59%   +0.01%     
==========================================
  Files         171      171              
  Lines        2473     2475       +2     
  Branches      444      445       +1     
==========================================
+ Hits         1968     1970       +2     
  Misses        388      388              
  Partials      117      117

Impacted Files	Coverage Δ
...c/components/Sidebar/components/NavItem/NavItem.js	100% <ø> (ø)	⬆️
src/util/shared-prop-types.js	92.59% <ø> (ø)	⬆️
src/components/ButtonGroup/ButtonGroup.js	95% <100%> (+0.55%)	⬆️

[ilyanoskov]

🎉 This PR is included in version 1.2.2-canary.6 🎉

The release is available on:

• npm package (@canary dist-tag)
• GitHub release

Your semantic-release bot 📦🚀

[ilyanoskov]

🎉 This PR is included in version 1.3.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀