wrapper: use longer throttleTime instead of shorter debounceTime to t…

…hrottle installedRepo observable (#315)
aragon · May 29, 2019 · 0caae25 · 0caae25
1 parent 783ad0e
commit 0caae25
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/packages/aragon-wrapper/src/index.js b/packages/aragon-wrapper/src/index.js
@@ -1,5 +1,5 @@
 // Externals
-import { concat, ReplaySubject, Subject, BehaviorSubject, merge, of } from 'rxjs'
+import { asyncScheduler, concat, merge, of, ReplaySubject, Subject, BehaviorSubject } from 'rxjs'
 import {
   concatMap,
   debounceTime,
@@ -15,6 +15,7 @@ import {
   startWith,
   switchMap,
   tap,
+  throttleTime,
   withLatestFrom
 } from 'rxjs/operators'
 import uuidv4 from 'uuid/v4'
@@ -770,7 +771,8 @@ export default class Aragon {
           return nextRepos
         }
       }, []),
-      debounceTime(100),
+      // Throttle updates, but must keep trailing to ensure we don't drop any updates
+      throttleTime(500, asyncScheduler, { leading: false, trailing: true }),
       publishReplay(1)
     )
     this.installedRepos.connect()