Merge pull request #14 from manosim/thinner-sidebar

New Sidebar

Author: manosim

main.js

@@ -162,6 +162,7 @@ app.on('ready', () => {
     checkAutoUpdate(false);
   }
 
+  app.dock.hide();
   appTrayIcon = createAppIcon();
   initWindow();
   positioner.move('trayCenter', appTrayIcon.getBounds());

src/js/actions/index.test.ts

@@ -630,62 +630,6 @@ describe('actions/index.js', () => {
       });
   });
 
-  it('should check if the user has starred the repository (has)', () => {
-    nock('https://api.github.com/')
-      .get(`/user/starred/${Constants.REPO_SLUG}`)
-      .reply(200);
-
-    const expectedActions = [
-      { type: actions.HAS_STARRED.REQUEST },
-      { type: actions.HAS_STARRED.SUCCESS, payload: '' },
-    ];
-
-    const store = createMockStore(
-      {
-        auth: { token: 'IAMATOKEN' },
-        settings: {
-          participating: false,
-          isEnterprise: false,
-          baseUrl: 'github.com',
-        },
-        notifications: { response: [] },
-      },
-      expectedActions
-    );
-
-    return store.dispatch(actions.checkHasStarred()).then(() => {
-      expect(store.getActions()).toEqual(expectedActions);
-    });
-  });
-
-  it('should check if the user has starred the repository (has not)', () => {
-    nock('https://api.github.com/')
-      .get(`/user/starred/${Constants.REPO_SLUG}`)
-      .reply(404);
-
-    const expectedActions = [
-      { type: actions.HAS_STARRED.REQUEST },
-      { type: actions.HAS_STARRED.FAILURE, payload: '' },
-    ];
-
-    const store = createMockStore(
-      {
-        auth: { token: 'IAMATOKEN' },
-        settings: {
-          participating: false,
-          isEnterprise: false,
-          baseUrl: 'github.com',
-        },
-        notifications: { response: [] },
-      },
-      expectedActions
-    );
-
-    return store.dispatch(actions.checkHasStarred()).then(() => {
-      expect(store.getActions()).toEqual(expectedActions);
-    });
-  });
-
   it('should search the notifications with a query', () => {
     const query = 'hello';
 

src/js/actions/index.ts

@@ -193,27 +193,6 @@ export function markRepoNotifications(repoSlug, hostname) {
   };
 }
 
-// Starred
-
-export const HAS_STARRED = makeAsyncActionSet('HAS_STARRED');
-export function checkHasStarred() {
-  return (dispatch, getState: () => AppState) => {
-    const url = `https://api.${Constants.DEFAULT_AUTH_OPTIONS.hostname}/user/starred/${Constants.REPO_SLUG}`;
-    const method = 'GET';
-    const token = getState().auth.token;
-
-    dispatch({ type: HAS_STARRED.REQUEST });
-
-    return apiRequestAuth(url, method, token)
-      .then(function(response) {
-        dispatch({ type: HAS_STARRED.SUCCESS, payload: response.data });
-      })
-      .catch(function(error) {
-        dispatch({ type: HAS_STARRED.FAILURE, payload: error.response.data });
-      });
-  };
-}
-
 // Search
 
 export const SEARCH_NOTIFICATIONS = 'SEARCH_NOTIFICATIONS';