Serubin / pulse-sms-web

The official web app for Pulse SMS - built on Vue.js.
https://pulsesms.app
Other
109 stars 44 forks source link

Don't push the login route, if we are already there #88

Closed klinker24 closed 5 years ago

klinker24 commented 5 years ago

This closes #85.

It happened because we were simply already on the /login route. When setting up the routing, if we don't have the account_id, it pushes to the /login route automatically.

I think we probably could remove this else branch completely, but there could be cases that I am not considering. I am not fully confident in it, so I would rather just leave it there with the condition.