First issue lint fix

[joemclo]

Hi,

I stumbled upon this repo whilst experiencing the #251 bug and saw that it is open for contributions.

So as a first issue this PR fixes two small lint issues that are reported as warnings when running yarn start command. If this is helpful I might continue contributing and fix some listed issues.

lint issues fixed in this PR

/src/App.js
  Line 153:5:  'PathWithSideMenu' is assigned a value but never used  no-unused-vars

./src/components/LocationBanner/LocationBanner.js
  Line 77:25:  Missing '()' invoking a constructor 

[lgtm-com]

This pull request introduces 1 alert when merging 75e4ec8 into 9a3ca57 - view on LGTM.com

new alerts:

• 1 for Unused variable, import, function or class

[xenatisch]

I really don't like the implementation of menu links. It's in the backlog to reimplement them as a more generic component, instead of adding in individual paths one at a time.