New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

clean up home.js #108

Open

karliatto opened this issue Jan 23, 2017 · 1 comment

Member

karliatto commented Jan 23, 2017

The variable loggedMessage is never used

var loggedMessage = 'You are logged';

It is located in Line 9 and Line 24 but is not anymore used in home.html.

https://github.com/Learn-by-doing/bitstarter/blob/master/controllers/home.js#L9

https://github.com/Learn-by-doing/bitstarter/blob/master/controllers/home.js#L24

Contributor

drurenia commented Jan 30, 2017 •

edited

Loading

I've changed it even though I was not aware of this issue. The clean up will be merged as soon as the pull request #109 is accepted.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment