Agreed, it's awkward. I'll rework that.

Matthew Riley MacPherson (Sent from mobile)

> On Nov 6, 2013, at 10:06, "sole" <notifications@github.com> wrote:
> 
> In www/js/views/app.coffee:
> 
> > @@ -43,14 +51,20 @@ define ['zepto', 'underscore', 'backbone', 'brick', 'cs!collections/users', 'tpl
> >        # prevents the "please sign in screen" from appearing during sign-in.
> >        return if @selfUser or window.location.hash.match 'access_token='
> >  
> > -      @selfUser = Users.getSelf()
> > -
> > -      # If there's no "selfUser", we need to display an intro screen/login
> > -      # prompt and authorize the user's device.
> > -      unless @selfUser
> > -        console.info "No user with relationship: RELATIONSHIP_SELF found"
> > -        # We manually set the location.hash here because the router hasn't
> > -        # actually finished loading yet (and thus isn't assigned to
> > -        # `window.router`, where we'd usually access its `.navigate`
> > -        # method).
> > -        window.location.hash = "login"
> > +      self = this
> > +
> > +      $.when(Users).done ->
> > +        self.selfUser = Users.getSelf()
> another minor nitpicky thing :-) what about calling this var self.user ? self.selfUser strikes me as too repetitive :)
> 
> —
> Reply to this email directly or view it on GitHub.