reactgraphqlacademy/fb-messenger

thinking in react branch: loggedUser is defined but never used

Closed this issue · 1 comments

loggedUser is defined as a variable on this line in App.js but this variable is not used anywhere

I'm guessing the intent was to use it on this line, where Alex is hardcoded?

Perhaps it's worth either removing the line that creates the loggedUser variable, or using it in the link:

<a
  onClick={this.toggleModal}
  className={`username ${this.userPosition || "left"}`}
>
  {loggedUser}
</a>

Just to save potential confusion during the refactor stages.

Fixed with #4 ?