#2758 Reduce default pagination limit to 30

linagora · Jul 22, 2019 · 301dcc7 · 301dcc7
1 parent 7892b37
commit 301dcc7
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 4 deletions.
diff --git a/frontend/js/modules/pagination/pagination-provider-builder.service.spec.js b/frontend/js/modules/pagination/pagination-provider-builder.service.spec.js
@@ -29,6 +29,7 @@ describe('The esnPaginationtionProviderBuilder factory', function() {
     module('esn.pagination', function($provide) {
       $provide.value('esnPaginationProvider', esnPaginationProviderMock);
       $provide.value('PageAggregatorService', PageAggregatorServiceMock);
+      $provide.constant('ELEMENTS_PER_PAGE', 1);
     });
   });
 

diff --git a/templates/js/constants.ejs b/templates/js/constants.ejs
@@ -7,8 +7,8 @@ angular.module('esn.constants', [])
 
   .constant('AUTOCOMPLETE_MAX_RESULTS', <%= getConstant('AUTOCOMPLETE_MAX_RESULTS', 5) %>)
 
-  .constant('ELEMENTS_PER_REQUEST', <%= getConstant('ELEMENTS_PER_REQUEST', 200) %>)
-  .constant('ELEMENTS_PER_PAGE', <%= getConstant('ELEMENTS_PER_PAGE', 1) %>)
+  .constant('ELEMENTS_PER_REQUEST', <%= getConstant('ELEMENTS_PER_REQUEST', 30) %>)
+  .constant('ELEMENTS_PER_PAGE', <%= getConstant('ELEMENTS_PER_PAGE', 30) %>)
 
   .constant('INFINITE_LIST_DISTANCE', <%= getConstant('INFINITE_LIST_DISTANCE', 0.5) %>)
   .constant('INFINITE_LIST_DISABLED', <%= getConstant('INFINITE_LIST_DISABLED', false) %>)

diff --git a/test/fixtures/code-generation/constants.js b/test/fixtures/code-generation/constants.js
@@ -7,8 +7,8 @@ angular.module('esn.constants', [])
 
   .constant('AUTOCOMPLETE_MAX_RESULTS', 5)
 
-  .constant('ELEMENTS_PER_REQUEST', 200)
-  .constant('ELEMENTS_PER_PAGE', 1)
+  .constant('ELEMENTS_PER_REQUEST', 30)
+  .constant('ELEMENTS_PER_PAGE', 30)
 
   .constant('INFINITE_LIST_DISTANCE', 0.5)
   .constant('INFINITE_LIST_DISABLED', false)