Remove custom LoggedInView::shouldComponentUpdate logic

[germain-gg]

This pull request reduces the work that needs to be done on every sync by approximately half. It scraps the second part of the call stack

LoggedInView::shouldComponentUpdate, would ALWAYS return true. And you can find the following code on the onSync callback

this.setState({
    syncErrorData: null,
});

The comment regarding why shouldComponentUpdate was written this way is also now irrelevant as whenever a logout occurs an action is dispatched and the LoggedInView is unmounted