From 005a5b59bedae3ccd2572421ef5811e99a0e6c30 Mon Sep 17 00:00:00 2001
From: Emmanouil Konstantinidis <manos@iamemmanouil.com>
Date: Sat, 27 Aug 2016 01:44:41 +0100
Subject: [PATCH 1/2] 186 - Hide "Star on GitHub"

---
 src/js/actions/index.js            | 21 +++++++++++++++++++++
 src/js/components/login.js         |  2 +-
 src/js/components/notifications.js |  3 ++-
 src/js/middleware/requests.js      |  8 +++++++-
 src/js/reducers/settings.js        | 17 +++++++++++++++--
 src/js/store/configureStore.js     | 13 ++++++++++---
 src/js/utils/constants.js          |  2 ++
 7 files changed, 58 insertions(+), 8 deletions(-)

diff --git a/src/js/actions/index.js b/src/js/actions/index.js
index c221fc66b..855c17ed2 100644
--- a/src/js/actions/index.js
+++ b/src/js/actions/index.js
@@ -115,6 +115,27 @@ export function markRepoNotifications(loginId, repoId, repoFullName) {
 };
 
 
+// Starred
+
+export const HAS_STARRED_REQUEST = 'HAS_STARRED_REQUEST';
+export const HAS_STARRED_SUCCESS = 'HAS_STARRED_SUCCESS';
+export const HAS_STARRED_FAILURE = 'HAS_STARRED_FAILURE';
+export function checkHasStarred() {
+  return {
+    [CALL_API]: {
+      endpoint: `https://api.github.com/user/starred/${Constants.REPO_SLUG}`,
+      method: 'GET',
+      headers: {
+        'Accept': 'application/json',
+        'Cache-Control': 'no-cache',
+        'Content-Type': 'application/json'
+      },
+      types: [HAS_STARRED_REQUEST, HAS_STARRED_SUCCESS, HAS_STARRED_FAILURE]
+    }
+  };
+};
+
+
 // Search
 
 export const SEARCH_NOTIFICATIONS = 'SEARCH_NOTIFICATIONS';
diff --git a/src/js/components/login.js b/src/js/components/login.js
index 09d4f9765..3d87b75e5 100644
--- a/src/js/components/login.js
+++ b/src/js/components/login.js
@@ -75,7 +75,7 @@ export class LoginPage extends React.Component {
     return (
       <div className="container-fluid main-container login">
         <div className="row">
-          <div className="col-xs-offset-2 col-xs-8">
+          <div className="offset-xs-2 col-xs-8">
             <img className="img-responsive logo" src="images/gitify-logo-outline-dark.png" />
             <div className="desc">GitHub Notifications<br />in your menu bar.</div>
             <button className="btn btn-lg btn-block" onClick={this.authGithub.bind(this)}>
diff --git a/src/js/components/notifications.js b/src/js/components/notifications.js
index 842ff5f14..0c764a0c0 100644
--- a/src/js/components/notifications.js
+++ b/src/js/components/notifications.js
@@ -62,7 +62,7 @@ export class NotificationsPage extends React.Component {
           })}
         </ReactCSSTransitionGroup>
 
-        {!_.isEmpty(groupedNotifications) ? (
+        {!_.isEmpty(groupedNotifications) && !this.props.hasStarred ? (
           <div className="fork" onClick={this.openBrowser}>
             <i className="fa fa-github" /> Star Gitify on GitHub
           </div>
@@ -74,6 +74,7 @@ export class NotificationsPage extends React.Component {
 
 function mapStateToProps(state) {
   return {
+    hasStarred: state.settings.hasStarred,
     failed: state.notifications.failed,
     notifications: state.notifications.response,
     searchQuery: state.searchFilter.query
diff --git a/src/js/middleware/requests.js b/src/js/middleware/requests.js
index 5d8160064..32f53e76e 100644
--- a/src/js/middleware/requests.js
+++ b/src/js/middleware/requests.js
@@ -1,5 +1,10 @@
 import { CALL_API, isRSAA } from 'redux-api-middleware';
-import { NOTIFICATIONS_REQUEST, MARK_NOTIFICATION_REQUEST, MARK_REPO_NOTIFICATION_REQUEST } from '../actions';
+import {
+  HAS_STARRED_REQUEST,
+  NOTIFICATIONS_REQUEST,
+  MARK_NOTIFICATION_REQUEST,
+  MARK_REPO_NOTIFICATION_REQUEST
+} from '../actions';
 
 export default store => next => action => {
   if (!isRSAA(action)) {
@@ -12,6 +17,7 @@ export default store => next => action => {
       const endpoint = action[CALL_API].endpoint + '?participating=';
       action[CALL_API].endpoint = endpoint + (settings.participating ? 'true' : 'false');
 
+    case HAS_STARRED_REQUEST:
     case NOTIFICATIONS_REQUEST:
     case MARK_NOTIFICATION_REQUEST:
     case MARK_REPO_NOTIFICATION_REQUEST:
diff --git a/src/js/reducers/settings.js b/src/js/reducers/settings.js
index 71796df55..7ff683e45 100644
--- a/src/js/reducers/settings.js
+++ b/src/js/reducers/settings.js
@@ -1,5 +1,7 @@
 import {
-  UPDATE_SETTING
+  UPDATE_SETTING,
+  HAS_STARRED_SUCCESS,
+  HAS_STARRED_FAILURE
 } from '../actions';
 
 const initialState = {
@@ -7,7 +9,8 @@ const initialState = {
   playSound: true,
   showNotifications: true,
   markOnClick: false,
-  openAtStartup: false
+  openAtStartup: false,
+  hasStarred: false
 };
 
 export default function reducer(state = initialState, action) {
@@ -17,6 +20,16 @@ export default function reducer(state = initialState, action) {
         ...state,
         [action.setting]: action.value
       };
+    case HAS_STARRED_SUCCESS:
+      return {
+        ...state,
+        hasStarred: true
+      };
+    case HAS_STARRED_FAILURE:
+      return {
+        ...state,
+        hasStarred: false
+      };
     default:
       return state;
   }
diff --git a/src/js/store/configureStore.js b/src/js/store/configureStore.js
index cf1eb4d21..da6165fd0 100644
--- a/src/js/store/configureStore.js
+++ b/src/js/store/configureStore.js
@@ -5,7 +5,7 @@ import * as storage from 'redux-storage';
 import createEngine from 'redux-storage-engine-localstorage';
 import filter from 'redux-storage-decorator-filter';
 
-import { fetchNotifications, UPDATE_SETTING, LOGIN_SUCCESS, LOGOUT } from '../actions';
+import { checkHasStarred, fetchNotifications, UPDATE_SETTING, LOGIN_SUCCESS, LOGOUT } from '../actions';
 import constants from '../utils/constants';
 import notifications from '../middleware/notifications';
 import settings from '../middleware/settings';
@@ -13,7 +13,11 @@ import requests from '../middleware/requests';
 import rootReducer from '../reducers';
 
 export default function configureStore(initialState) {
-  const engine = filter(createEngine(constants.STORAGE_KEY), ['settings', ['auth', 'token']]);
+  const engine = filter(
+    createEngine(constants.STORAGE_KEY),
+    ['settings', ['auth', 'token']],
+    [['settings', 'hasStarred']]
+  );
   const storageMiddleware = storage.createMiddleware(engine, [], [UPDATE_SETTING, LOGIN_SUCCESS, LOGOUT]);
 
   const createStoreWithMiddleware = applyMiddleware(
@@ -32,7 +36,10 @@ export default function configureStore(initialState) {
     .then(function (newState) {
       // Check if the user is logged in
       const isLoggedIn = store.getState().auth.token !== null;
-      if (isLoggedIn) { store.dispatch(fetchNotifications()); }
+      if (isLoggedIn) {
+        store.dispatch(checkHasStarred());
+        store.dispatch(fetchNotifications());
+      }
     });
 
   return store;
diff --git a/src/js/utils/constants.js b/src/js/utils/constants.js
index d9b15a218..e84c0c6dd 100644
--- a/src/js/utils/constants.js
+++ b/src/js/utils/constants.js
@@ -4,6 +4,8 @@ let constants = {
   CLIENT_SECRET: '9670de733096c15322183ff17ed0fc8704050379',
   SCOPE: ['user:email', 'notifications'],
 
+  REPO_SLUG: 'ekonstantinidis/gitify',
+
   // Storage
   STORAGE_KEY: 'gitify-storage',
 

From 820458e160939bd8f435cbb8a8f4181992507582 Mon Sep 17 00:00:00 2001
From: Emmanouil Konstantinidis <manos@iamemmanouil.com>
Date: Sat, 27 Aug 2016 01:56:14 +0100
Subject: [PATCH 2/2] Tests

---
 src/js/__tests__/actions/index.js     | 43 +++++++++++++++++++++++++--
 src/js/__tests__/reducers/settings.js | 32 ++++++++++++++++++--
 2 files changed, 71 insertions(+), 4 deletions(-)

diff --git a/src/js/__tests__/actions/index.js b/src/js/__tests__/actions/index.js
index 33b206b7c..d76a2543b 100644
--- a/src/js/__tests__/actions/index.js
+++ b/src/js/__tests__/actions/index.js
@@ -1,7 +1,9 @@
 import { expect } from 'chai';
 import nock from 'nock';
-import { apiMiddleware } from 'redux-api-middleware';
+import { apiMiddleware, ApiError } from 'redux-api-middleware';
 import configureMockStore from 'redux-mock-store';
+
+import Constants from '../../utils/constants';
 import * as actions from '../../actions';
 
 const middlewares = [ apiMiddleware ];
@@ -186,7 +188,6 @@ describe('actions/index.js', () => {
 
   });
 
-
   it('should mark a repository\'s notifications as read with success', () => {
     const loginId = 'ekonstantinidis';
     const repoId = 'gitify';
@@ -249,6 +250,44 @@ describe('actions/index.js', () => {
 
   });
 
+  it('should check if the user has starred the repository', () => {
+    nock('https://api.github.com/')
+      .get(`/user/starred/${Constants.REPO_SLUG}`)
+      .reply(200);
+
+    const expectedActions = [
+      { type: actions.HAS_STARRED_REQUEST, payload: undefined, meta: undefined },
+      { type: actions.HAS_STARRED_SUCCESS, payload: undefined, meta: undefined }
+    ];
+
+    const store = createMockStore({ response: [] }, expectedActions);
+
+    return store.dispatch(actions.checkHasStarred())
+      .then(() => { // return of async actions
+        expect(store.getActions()).to.eql(expectedActions);
+      });
+
+  });
+
+  it('should check if the user has starred the repository', () => {
+    nock('https://api.github.com/')
+      .get(`/user/starred/${Constants.REPO_SLUG}`)
+      .reply(404);
+
+    const apiError = new ApiError(404, 'Not Found', undefined);
+    const expectedActions = [
+      { type: actions.HAS_STARRED_REQUEST, payload: undefined, meta: undefined },
+      { type: actions.HAS_STARRED_FAILURE, payload: apiError, error: true, meta: undefined }
+    ];
+
+    const store = createMockStore({ response: [] }, expectedActions);
+
+    return store.dispatch(actions.checkHasStarred())
+      .then(() => { // return of async actions
+        expect(store.getActions()).to.eql(expectedActions);
+      });
+
+  });
 
   it('should search the notifications with a query', () => {
 
diff --git a/src/js/__tests__/reducers/settings.js b/src/js/__tests__/reducers/settings.js
index 4d143554f..300931e9f 100644
--- a/src/js/__tests__/reducers/settings.js
+++ b/src/js/__tests__/reducers/settings.js
@@ -1,6 +1,6 @@
 import { expect } from 'chai';
 import reducer from '../../reducers/settings';
-import { UPDATE_SETTING } from '../../actions';
+import { UPDATE_SETTING, HAS_STARRED_SUCCESS, HAS_STARRED_FAILURE } from '../../actions';
 
 describe('reducers/settings.js', () => {
   const initialState = {
@@ -8,7 +8,8 @@ describe('reducers/settings.js', () => {
     playSound: true,
     showNotifications: true,
     markOnClick: false,
-    openAtStartup: false
+    openAtStartup: false,
+    hasStarred: false
   };
 
   it('should return the initial state', () => {
@@ -42,4 +43,31 @@ describe('reducers/settings.js', () => {
     });
 
   });
+
+  it('should handle HAS_STARRED_SUCCESS', () => {
+
+    const actionParticipating = {
+      type: HAS_STARRED_SUCCESS
+    };
+
+    expect(reducer(undefined, actionParticipating)).to.eql({
+      ...initialState,
+      hasStarred: true
+    });
+
+  });
+
+  it('should handle HAS_STARRED_SUCCESS', () => {
+
+    const actionParticipating = {
+      type: HAS_STARRED_FAILURE
+    };
+
+    expect(reducer(undefined, actionParticipating)).to.eql({
+      ...initialState,
+      hasStarred: false
+    });
+
+  });
+
 });