fixed:#246

[saikat041]

Fixes #246

Checklist

• [x] I have read the Contribution & Best practices Guide and my PR follows them.
• [x] My branch is up-to-date with the Upstream master branch.
• [x] The unit tests pass locally with my changes
• [x] I have added tests that prove my fix is effective or that my feature works
• [x] I have added necessary documentation (if appropriate)
• [x] All the functions created/modified in this PR contain relevant docstrings.

Short description of what this resolves:

Now the error in logging in is shown using bootstrap alert method.

Changes proposed in this pull request:

-

-

[ms10398]

Please squash commits

[saikat041]

@ms10398 please review it

[saikat041]

@ms10398 now its done

[saikat041]

@ms10398 please review it

[ms10398]

@gabru-md @stealthanthrax @sachincool Please review

[ms10398]

Yeah you should add it too and please reference issue no. in your pull request.

[saikat041]

@gabru-md But after successful log in the page is changed. And showing "You are logged in " in that page is unnecessary i think.

[sansyrox]

@saikat041 , you can add "you are logged in as: email" for the page

[gabru-md]

@saikat041 i get your point. So I guess we better change the alert text from alert(response.message) to something more relevant. Can you please do it? something like : alert("Logged in!") ?

[gabru-md]

I guess I did not see that! It already says Logged in as : email. @ms10398 can you confirm if I am right ?

[ms10398]

Yes its there @gabru-md

[saikat041]

Can we use this instead of alert("You are logged in as email")

[gabru-md]

Sure @saikat041 Please commit it and squash your commits so that it can be merged!

[saikat041]

Now all error are shown using bootstrap alert method.