sr-npm 1.7.1079 → 1.7.1086

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sr-npm",
-  "version": "1.7.1079",
+  "version": "1.7.1086",
   "description": "",
   "main": "index.js",
   "scripts": {

package/pages/careersMultiBoxesPage.js

@@ -21,6 +21,7 @@ let allsecondarySearchJobs=[] // secondary search results that are displayed in
 let currentSecondarySearchJobs=[] // current secondary search results that are displayed in the jobs repeater
 let secondarySearchIsFilled=false // whether the secondary search is filled with results
 let keywordAllJobs; // all jobs that are displayed in the jobs repeater when the keyword is filled
+let dontActivateURLOnchange=false; // whether to activate the url onchange
 const pagination = {
   pageSize: 10,
   currentPage: 1,
@@ -33,7 +34,7 @@ async function careersMultiBoxesPageOnReady(_$w,urlParams) {
   });
 
   await loadData(_$w);
-  await loadJobsRepeater(_$w);
+  await loadJobsRepeater(_$w); // if we remove the await here the job list will be flaky , it doesn't fill it properly
   loadPrimarySearchRepeater(_$w);
   await loadFilters(_$w);
   loadSelectedValuesRepeater(_$w);
@@ -52,13 +53,15 @@ async function careersMultiBoxesPageOnReady(_$w,urlParams) {
 }
 
 async function handleBackAndForth(_$w){
+  console.log("inside handleBackAndForth dontActivateURLOnchange: ",dontActivateURLOnchange);
+  if(!dontActivateURLOnchange) {
     const newQueryParams=await location.query();
-    console.log("newQueryParams: ", newQueryParams);
-    await clearAll(_$w,true);
-    await handleUrlParams(_$w,newQueryParams); 
+      console.log("newQueryParams: ", newQueryParams);
+      await clearAll(_$w,true);
+      await handleUrlParams(_$w,newQueryParams); 
+    }
 }
 
-
 async function clearAll(_$w,urlOnChange=false) {
   if(selectedByField.size>0 || _$w(CAREERS_MULTI_BOXES_PAGE_CONSTS.SECONDARY_SEARCH_INPUT).value || _$w(CAREERS_MULTI_BOXES_PAGE_CONSTS.PRIMARY_SEARCH_INPUT).value) {
     for(const field of allfields) {
@@ -238,6 +241,7 @@ async function handleParams(_$w,param,values) {
             const field=getFieldById(fieldId,allfields);
             let fieldTitle=field.title.toLowerCase().replace(' ', '');
             fieldTitle==="brands"? fieldTitle="brand":fieldTitle;
+            dontActivateURLOnchange=true;
             if (updated.length) {
               selectedByField.set(fieldId, updated);
               queryParams.add({ [fieldTitle] : updated.map(val=>encodeURIComponent(val)).join(',') });
@@ -245,6 +249,7 @@ async function handleParams(_$w,param,values) {
               selectedByField.delete(fieldId);
               queryParams.remove([fieldTitle ]);
             }
+            dontActivateURLOnchange=false;
             const currentVals = _$w(`#${FiltersIds[field.title]}CheckBox`).value || [];
             const nextVals = currentVals.filter(v => v !== valueId);
             _$w(`#${FiltersIds[field.title]}CheckBox`).value = nextVals;
@@ -344,7 +349,7 @@ async function loadJobsRepeater(_$w) {
         const selected = ev.target.value; // array of selected value IDs
         let fieldTitle=field.title.toLowerCase().replace(' ', '');
         fieldTitle==="brands"? fieldTitle="brand":fieldTitle;
-
+        dontActivateURLOnchange=true;
         console.log("selected: ",selected)
         if (selected && selected.length) {
           selectedByField.set(field._id, selected); 
@@ -356,14 +361,13 @@ async function loadJobsRepeater(_$w) {
           else{
             queryParams.add({ [fieldTitle] : selected.map(val=>encodeURIComponent(val)).join(',') });
           }
-          
+          dontActivateURLOnchange=false;
         } else {
           selectedByField.delete(field._id);  
           queryParams.remove([fieldTitle ]);
         }
         console.log("selectedByField: ",selectedByField)
         await updateJobsAndNumbersAndFilters(_$w);
-    
       });
       const runFilter = debounce(() => {
       const query = (_$w(`#${FiltersIds[field.title]}input`).value || '').toLowerCase().trim();