Merge pull request #1006 from hypothesis/replace-q-references

Replace a usage of AngularJS promises ($q)

Merge pull request #1006 from hypothesis/replace-q-references
Replace a usage of AngularJS promises ($q)
d30f27ad · Robert Knight · GitHub · a3c6dd46 · ec0e043b · d30f27ad
Unverified Commit d30f27ad authored Mar 29, 2019 by Robert Knight Committed by GitHub Mar 29, 2019
Show whitespace changes
Inline Side-by-side

Showing with 1 addition and 2 deletions

session.js src/sidebar/services/session.js +1 -2

No files found.
--- a/src/sidebar/services/session.js
+++ b/src/sidebar/services/session.js
@@ -21,7 +21,6 @@ const CACHE_TTL = 5 * 60 * 1000; // 5 minutes
 * @ngInject
 */
 function session(
-  $q,
  $rootScope,
  analytics,
  store,
@@ -148,7 +147,7 @@ function session(
      .catch(function(err) {
        flash.error('Log out failed');
        analytics.track(analytics.events.LOGOUT_FAILURE);
-        return $q.reject(new Error(err));
+        throw new Error(err);
      })
      .then(function() {
        analytics.track(analytics.events.LOGOUT_SUCCESS);