chore: remove issues reducers (#51)

Author: Spring3

render/actions/issues.actions.js

@@ -3,45 +3,11 @@ import moment from 'moment';
 
 import request, { notify } from './helper';
 
-export const ISSUES_GET_PAGE = 'ISSUES_GET_PAGE';
 export const ISSUES_GET = 'ISSUES_GET';
 export const ISSUES_COMMENTS_SEND = 'ISSUES_COMMENTS_SEND';
 export const ISSUES_RESET_SELECTION = 'ISSUES_RESET_SELECTION';
 export const ISSUES_TIME_ENTRY_GET = 'ISSUES_TIME_ENTRY_GET';
 
-const getPage = (filter, pageNumber, batchSize) => (dispatch, getState) => {
-  const { issues } = getState();
-  const limit = typeof batchSize === 'number' ? Math.abs(batchSize) : issues.all.limit;
-  const page = typeof pageNumber === 'number' && pageNumber >= 0 ? pageNumber : issues.all.page;
-  const offset = page * limit;
-  let query = {
-    include: 'attachments,children,relations,journals',
-    offset,
-    limit
-  };
-
-  if (filter) {
-    query = {
-      ...query,
-      ...filter
-    };
-  }
-
-  dispatch(notify.start(ISSUES_GET_PAGE, { page }));
-
-  return request({
-    url: '/issues.json',
-    id: `getIssues:${page}`,
-    query
-  })
-    .then(({ data }) => dispatch(notify.ok(ISSUES_GET_PAGE, data, { page })))
-    .catch((error) => {
-      // eslint-disable-next-line
-      console.error('Error when trying to get a list of issues:', error.message);
-      dispatch(notify.nok(ISSUES_GET_PAGE, error, { page }));
-    });
-};
-
 const get = (id) => (dispatch) => {
   dispatch(notify.start(ISSUES_GET));
 
@@ -126,7 +92,6 @@ const getTimeEntriesPage = (issueId, projectId, pageNumber, batchSize) => (dispa
 const resetSelected = () => ({ type: ISSUES_RESET_SELECTION });
 
 export default {
-  getPage,
   get,
   getTimeEntriesPage,
   sendComments,

render/reducers/__tests__/issues.reducer.spec.js

@@ -1,6 +1,5 @@
 import { combineReducers } from 'redux';
 import settingsReducer from './settings.reducer';
-import allIssuesReducer from './issues.reducer';
 import issueReducer from './issue.reducer';
 import selectedIssueReducer from './issue.selected.reducer';
 import trackingReducer from './tracking.reducer';
@@ -10,7 +9,6 @@ import timeEntryReducer from './timeEntry.reducer';
 const appReducer = combineReducers({
   settings: settingsReducer,
   issues: combineReducers({
-    all: allIssuesReducer,
     selected: selectedIssueReducer
   }),
   issue: issueReducer,

render/reducers/index.js

@@ -10,7 +10,7 @@ import { ThemeProvider } from 'styled-components';
 
 import { theme } from '../../../theme';
 import { SummaryPage } from '../../AppViewPages/SummaryPage';
-import issueActions, { ISSUES_GET_PAGE } from '../../../actions/issues.actions';
+import issueActions from '../../../actions/issues.actions';
 import { initialize, getInstance, reset } from '../../../../common/request';
 
 const mockStore = configureStore([thunk]);
@@ -70,7 +70,6 @@ describe('AppView -> Summary Page', () => {
     );
 
     expect(spy).toHaveBeenCalled();
-    expect(store.getActions()[0].type).toBe(ISSUES_GET_PAGE);
     expect(store.getActions()[0].info.page).toBe(0);
     spy.mockRestore();
   });
@@ -116,7 +115,6 @@ describe('AppView -> Summary Page', () => {
 
     setTimeout(() => {
       expect(store.getActions().length).toBe(2);
-      expect(store.getActions().filter((action) => action.type === ISSUES_GET_PAGE).length).toBe(2);
       done();
     }, 1);
   });
@@ -157,7 +155,6 @@ describe('AppView -> Summary Page', () => {
     fireEvent.click(getAllByText('Id').pop());
     setTimeout(() => {
       expect(store.getActions().length).toBe(2);
-      expect(store.getActions().filter((action) => action.type === ISSUES_GET_PAGE).length).toBe(2);
       done();
     }, 1);
   });
@@ -198,7 +195,6 @@ describe('AppView -> Summary Page', () => {
     fireEvent.click(document.querySelector('#queryOptions input:first-child'));
     setTimeout(() => {
       expect(store.getActions().length).toBe(3);
-      expect(store.getActions().filter((action) => action.type === ISSUES_GET_PAGE).length).toBe(2);
       done();
     }, 1);
   });