From 4982260b4ed6e5a2ef2f8a89bac5d5688bf28e94 Mon Sep 17 00:00:00 2001
From: Jeff Daley <jeffdaley@users.noreply.github.com>
Date: Tue, 5 Mar 2024 13:52:07 -0500
Subject: [PATCH] Improve doc-owner link params (#628)

* Fix non-top-100 owner filtering

* Update toolbar.ts

* Add default query to TileMedium

* Improve doc-owner linkTo

* Fix broken tests, improve "Clear all" function
---
 web/app/components/doc/tile-medium.hbs |  4 ++--
 web/app/components/doc/tile-medium.ts  | 13 +++++++++++++
 2 files changed, 15 insertions(+), 2 deletions(-)

diff --git a/web/app/components/doc/tile-medium.hbs b/web/app/components/doc/tile-medium.hbs
index 61a389f5d..96bf28248 100644
--- a/web/app/components/doc/tile-medium.hbs
+++ b/web/app/components/doc/tile-medium.hbs
@@ -16,7 +16,7 @@
         data-test-document-owner-avatar
         tabindex="-1"
         @route="authenticated.documents"
-        @query={{hash owners=(array owner)}}
+        @query={{this.ownerQuery}}
         disabled={{not owner}}
       >
         <Person::Avatar
@@ -65,7 +65,7 @@
               tabindex="-1"
               data-test-document-owner-name
               @route="authenticated.documents"
-              @query={{hash owners=(array owner)}}
+              @query={{this.ownerQuery}}
               disabled={{not owner}}
               class="underline-on-hover inline-block"
             >
diff --git a/web/app/components/doc/tile-medium.ts b/web/app/components/doc/tile-medium.ts
index 6abd5d15c..9acb29344 100644
--- a/web/app/components/doc/tile-medium.ts
+++ b/web/app/components/doc/tile-medium.ts
@@ -4,6 +4,7 @@ import { HermesDocument } from "hermes/types/document";
 import { inject as service } from "@ember/service";
 import FetchService from "hermes/services/fetch";
 import ConfigService from "hermes/services/config";
+import { DEFAULT_FILTERS } from "hermes/services/active-filters";
 
 interface DocTileMediumComponentSignature {
   Element: HTMLAnchorElement;
@@ -63,6 +64,18 @@ export default class DocTileMediumComponent extends Component<DocTileMediumCompo
       return this.args.doc._snippetResult?.content.value;
     }
   }
+  /**
+   * The query parameters for the doc-owner LinkTos.
+   * Sets the "owners" filter to the owner of the document,
+   * and default values for the rest.
+   */
+  protected get ownerQuery() {
+    return {
+      ...DEFAULT_FILTERS,
+      owners: this.args.doc.owners,
+      page: 1,
+    };
+  }
 }
 
 declare module "@glint/environment-ember-loose/registry" {