hotfix: workflow list - rewrite arrays in url search params to remove items (#2734)

## Changes

- Deletes and rewrites arrays in URL search params in workflow list when
editing array filters (i.e. tags & profiles)
- Removes a missed `console.log`
- bump to 1.17.3

cc @SuaYoo

---------

Co-authored-by: Ilya Kreymer <ikreymer@gmail.com>

backend/btrixcloud/version.py

@@ -1,3 +1,3 @@
 """current version"""
 
-__version__ = "1.17.2"
+__version__ = "1.17.3"

chart/Chart.yaml

@@ -5,7 +5,7 @@ type: application
 icon: https://webrecorder.net/assets/icon.png
 
 # Browsertrix and Chart Version
-version: v1.17.2
+version: v1.17.3
 
 dependencies:
   - name: btrix-admin-logging

chart/values.yaml

@@ -106,7 +106,7 @@ replica_deletion_delay_days: 0
 
 # API Image
 # =========================================
-backend_image: "docker.io/webrecorder/browsertrix-backend:1.17.2"
+backend_image: "docker.io/webrecorder/browsertrix-backend:1.17.3"
 backend_pull_policy: "IfNotPresent"
 
 backend_password_secret: "PASSWORD!"
@@ -164,7 +164,7 @@ backend_avg_memory_threshold: 95
 
 # Nginx Image
 # =========================================
-frontend_image: "docker.io/webrecorder/browsertrix-frontend:1.17.2"
+frontend_image: "docker.io/webrecorder/browsertrix-frontend:1.17.3"
 frontend_pull_policy: "IfNotPresent"
 
 frontend_cpu: "10m"

frontend/package.json

@@ -1,6 +1,6 @@
 {
   "name": "browsertrix-frontend",
-  "version": "1.17.2",
+  "version": "1.17.3",
   "main": "index.ts",
   "license": "AGPL-3.0-or-later",
   "dependencies": {

frontend/src/features/crawl-workflows/workflow-tag-filter.ts

@@ -106,8 +106,6 @@ export class WorkflowTagFilter extends BtrixElement {
         @sl-after-hide=${() => {
           this.searchString = "";
 
-          console.log("after hide");
-
           this.dispatchEvent(
             new CustomEvent<
               BtrixChangeEvent<ChangeWorkflowTagEventDetails>["detail"]

frontend/src/pages/org/workflows-list.ts

@@ -354,11 +354,13 @@ export class WorkflowsList extends BtrixElement {
         for (const [filter, value] of newParams) {
           if (value !== undefined) {
             if (Array.isArray(value)) {
+              // Rather than a more efficient method where we compare the existing & wanted arrays,
+              // it's simpler to just delete and re-append values here. If we were working with large
+              // arrays, we could change this, but we'll leave it as is for now — if we were working
+              // with truly large arrays, we wouldn't be using search params anyways.
+              params.delete(filter);
               value.forEach((v) => {
-                // Only add new array values to URL
-                if (!params.getAll(filter).includes(v)) {
-                  params.append(filter, v);
-                }
+                params.append(filter, v);
               });
             } else {
               params.set(filter, value.toString());

version.txt

@@ -1 +1 @@
-1.17.2
+1.17.3