Merge pull request #117 from blackjackkent/100ThreadWarning

Remove pop up notification for more than 100 threads

Author: blackjackkent

src/infrastructure/sagas/__tests__/toastrSaga.spec.js

@@ -17,50 +17,6 @@ beforeEach(() => {
 	jest.resetAllMocks();
 });
 describe('saga behavior', () => {
-	describe('FETCHED_ACTIVE_THREADS_SUCCESS', () => {
-		it('should display a warning if user has over 100 threads', () => {
-			const threads = [];
-			for (let i = 0; i < 101; i++) {
-				threads.push({});
-			}
-			const action = {
-				type: actions.FETCHED_ACTIVE_THREADS_SUCCESS,
-				data: { threads }
-			};
-			const saga = new SagaTestWrapper(toastrSaga);
-			return saga.execute(action)
-				.then(() => {
-					expect(toastr.light).toHaveBeenCalledTimes(1);
-					expect(toastr.light).toHaveBeenLastCalledWith('Retrieving more than 100 threads; loading may take several minutes. Archive some threads to reduce loading time.', { status: 'info' });
-				});
-		});
-		it('should display no warning if user has 100 threads or less', () => {
-			const threads = [];
-			for (let i = 0; i < 100; i++) {
-				threads.push({});
-			}
-			const action = {
-				type: actions.FETCHED_ACTIVE_THREADS_SUCCESS,
-				data: { threads }
-			};
-			const saga = new SagaTestWrapper(toastrSaga);
-			return saga.execute(action)
-				.then(() => {
-					expect(toastr.light).toHaveBeenCalledTimes(0);
-				});
-		});
-		it('should display no warning if action has no data', () => {
-			const action = {
-				type: actions.FETCHED_ACTIVE_THREADS_SUCCESS,
-				data: null
-			};
-			const saga = new SagaTestWrapper(toastrSaga);
-			return saga.execute(action)
-				.then(() => {
-					expect(toastr.light).toHaveBeenCalledTimes(0);
-				});
-		});
-	});
 	describe('UNTRACK_THREAD_SUCCESS', () => {
 		it('should display success message', () => {
 			const action = {

src/infrastructure/sagas/toastrSaga.js

@@ -3,7 +3,6 @@ import { takeEvery, all } from 'redux-saga/effects';
 import { toastr } from 'react-redux-toastr';
 
 import {
-	FETCHED_ACTIVE_THREADS_SUCCESS,
 	UNTRACK_THREAD_SUCCESS,
 	UNTRACK_THREAD_FAILURE,
 	UPSERT_THREAD_FAILURE,
@@ -25,13 +24,6 @@ import {
 	EXPORT_THREADS_FAILURE
 } from '../actions';
 
-function* displayActiveThreadsCountMessage(action) {
-	const threadData = action.data;
-	if (threadData && threadData.threads && threadData.threads.length > 100) {
-		toastr.light('Retrieving more than 100 threads; loading may take several minutes. Archive some threads to reduce loading time.', { status: 'info' });
-	}
-}
-
 function* displayUntrackThreadError() {
 	toastr.error('There was a problem untracking your thread.');
 }
@@ -135,7 +127,6 @@ function* displayExportThreadsError() {
 
 export default function* fetchActiveThreadsSaga() {
 	yield all([
-		takeEvery(FETCHED_ACTIVE_THREADS_SUCCESS, displayActiveThreadsCountMessage),
 		takeEvery(UNTRACK_THREAD_SUCCESS, displayUntrackThreadSuccess),
 		takeEvery(UNTRACK_THREAD_FAILURE, displayUntrackThreadError),
 		takeEvery(UPSERT_THREAD_FAILURE, displayUpdateThreadError),