From e33af8a672869cf70b1c73d0109627fbf75ce9ad Mon Sep 17 00:00:00 2001 From: Piotr Sarnacki Date: Wed, 4 Feb 2015 19:47:38 +0100 Subject: [PATCH] Go one step further with lowering pusher data landing into store If user switches to recent repos tab and then switches away, there's really no point in pushing everything into store. We can still reject all of the events for repos that don't belong to the current user and are not yet in the store. --- app/routes/main/recent.coffee | 5 ++++- app/store.coffee | 2 +- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/app/routes/main/recent.coffee b/app/routes/main/recent.coffee index 5e8e2e2f..0d0ff136 100644 --- a/app/routes/main/recent.coffee +++ b/app/routes/main/recent.coffee @@ -5,6 +5,9 @@ Route = MainTabRoute.extend reposTabName: 'recent' activate: -> - @store.set('recentReposOpened', true) + @store.set('recentReposTabIsOpened', true) + + deactivate: -> + @store.set('recentReposTabIsOpened', true) `export default Route` diff --git a/app/store.coffee b/app/store.coffee index d5c40077..75267450 100644 --- a/app/store.coffee +++ b/app/store.coffee @@ -9,7 +9,7 @@ Store = DS.Store.extend [name, type] = event.split(':') auth = @container.lookup('auth:main') - if !@get('recentReposOpened') && event != 'job:log' && auth.get('signedIn') && + if !@get('recentReposTabIsOpened') && event != 'job:log' && auth.get('signedIn') && !config.pro && !config.enterprise # if recent repos hasn't been opened yet, we can safely # drop any events that doesn't belong to repos owned by