From 2218b8f5c0fafc8c804976f9589f11e292620366 Mon Sep 17 00:00:00 2001
From: Stanislav Romanov <kaermorchen@gmail.com>
Date: Thu, 15 Sep 2022 18:03:56 +0300
Subject: [PATCH] Add limit param for tasks list

---
 app/controllers/admin/tasks/index.js          |  13 ++
 app/routes/admin/tasks/index.js               |  16 +-
 app/services/meilisearch.js                   |   7 +-
 app/templates/admin/tasks/index.hbs           | 143 +++++++++++-------
 .../controllers/admin/tasks/index-test.js     |  12 ++
 5 files changed, 132 insertions(+), 59 deletions(-)
 create mode 100644 app/controllers/admin/tasks/index.js
 create mode 100644 tests/unit/controllers/admin/tasks/index-test.js

diff --git a/app/controllers/admin/tasks/index.js b/app/controllers/admin/tasks/index.js
new file mode 100644
index 0000000..ed14fcd
--- /dev/null
+++ b/app/controllers/admin/tasks/index.js
@@ -0,0 +1,13 @@
+import Controller from '@ember/controller';
+import { action } from '@ember/object';
+import { tracked } from '@glimmer/tracking';
+
+export default class AdminTasksIndexController extends Controller {
+  @tracked limit = 20;
+  @tracked from;
+  @tracked status;
+  @tracked type;
+  @tracked indexUid;
+
+  queryParams = ['limit', 'from', 'status', 'type', 'indexUid'];
+}
diff --git a/app/routes/admin/tasks/index.js b/app/routes/admin/tasks/index.js
index 09d0698..e1337d2 100644
--- a/app/routes/admin/tasks/index.js
+++ b/app/routes/admin/tasks/index.js
@@ -1,10 +1,22 @@
 import Route from '@ember/routing/route';
 import { inject as service } from '@ember/service';
+import { hash } from 'rsvp';
 
 export default class AdminTasksIndexRoute extends Route {
   @service meilisearch;
 
-  model() {
-    return this.meilisearch.getTasks();
+  queryParams = {
+    limit: { refreshModel: true },
+    from: { refreshModel: true },
+    status: { refreshModel: true },
+    type: { refreshModel: true },
+    indexUid: { refreshModel: true },
+  };
+
+  model(params) {
+    return hash({
+      tasks: this.meilisearch.getTasks(params),
+      indexes: this.meilisearch.getIndexes(),
+    });
   }
 }
diff --git a/app/services/meilisearch.js b/app/services/meilisearch.js
index 28deb7a..e964618 100644
--- a/app/services/meilisearch.js
+++ b/app/services/meilisearch.js
@@ -40,8 +40,11 @@ export default class MeilisearchService extends Service {
     return query(`keys/${uid}`);
   }
 
-  getTasks() {
-    return query('tasks');
+  getTasks(params) {
+    // Clear from undefined fields
+    Object.keys(params).forEach((key) => params[key] ?? delete params[key]);
+
+    return query(`tasks?${new URLSearchParams(params)}`);
   }
 
   getTask(uid) {
diff --git a/app/templates/admin/tasks/index.hbs b/app/templates/admin/tasks/index.hbs
index 51ea536..14c3b59 100644
--- a/app/templates/admin/tasks/index.hbs
+++ b/app/templates/admin/tasks/index.hbs
@@ -1,58 +1,91 @@
 <Headers::H1 @title='Tasks' />
 
-<RoyalTable @data={{@model.results}} class='w-full' as |Column data|>
-  <Column @prop='uid'>
-    <LinkTo @route='admin.tasks.task' @model={{data.uid}}>{{data.uid}}</LinkTo>
-  </Column>
-  <Column @prop='indexUid'>
-    <LinkTo @route='admin.indexes.item' @model={{data.indexUid}}>{{data.indexUid}}</LinkTo>
-  </Column>
-  <Column @prop='status'>
-    <Badge @label={{data.status}} @style='success' />
-  </Column>
-  <Column @prop='type' />
-  <Column @prop='details'>
-    {{json-stringify data.details}}
-  </Column>
-  <Column @prop='error'>
-    {{json-stringify data.error}}
-  </Column>
-  <Column @prop='duration' />
-  <Column @prop='enqueuedAt'>
-    {{date-time-format
-      data.enqueuedAt
-      year='numeric'
-      month='numeric'
-      day='numeric'
-      hour='numeric'
-      minute='numeric'
-      second='numeric'
-    }}
-  </Column>
-  <Column @prop='startedAt'>
-    {{date-time-format
-      data.startedAt
-      year='numeric'
-      month='numeric'
-      day='numeric'
-      hour='numeric'
-      minute='numeric'
-      second='numeric'
-    }}
-  </Column>
-  <Column @prop='finishedAt'>
-    {{date-time-format
-      data.finishedAt
-      year='numeric'
-      month='numeric'
-      day='numeric'
-      hour='numeric'
-      minute='numeric'
-      second='numeric'
-    }}
-  </Column>
-</RoyalTable>
-
-<div class='flex items-center'>
-
+<div class='sm:flex sm:flex-nowrap'>
+  <div class='relative mb-4 sm:ml-4 sm:flex-none'>
+    <Listbox @value={{this.limit}} @onChange={{set this 'limit'}} as |listbox|>
+      <listbox.Button
+        class='inline-flex items-center px-3 py-2 border rounded cursor-pointer'
+      >
+        Limit:
+        {{this.limit}}
+      </listbox.Button>
+      <listbox.Options
+        class='absolute z-dropdown right-0 w-28 min-w-min mt-2 py-1 origin-top-right bg-base-7 border rounded shadow-sm outline-none'
+        as |options|
+      >
+        {{#each (array 20 50 100 500) as |item|}}
+          <options.Option @value={{item}} class='outline-none'>
+            <div
+              class='flex items-center px-4 py-2 text-sm hover:bg-base-6 hover:text-base-1 cursor-pointer'
+            >
+              {{item}}
+            </div>
+          </options.Option>
+        {{/each}}
+      </listbox.Options>
+    </Listbox>
+  </div>
 </div>
+
+{{#if @model.tasks.results.length}}
+  <RoyalTable @data={{@model.tasks.results}} class='w-full' as |Column data|>
+    <Column @prop='uid'>
+      <LinkTo
+        @route='admin.tasks.task'
+        @model={{data.uid}}
+      >{{data.uid}}</LinkTo>
+    </Column>
+    <Column @prop='indexUid'>
+      <LinkTo
+        @route='admin.indexes.item'
+        @model={{data.indexUid}}
+      >{{data.indexUid}}</LinkTo>
+    </Column>
+    <Column @prop='status'>
+      <Badge @label={{data.status}} @style='success' />
+    </Column>
+    <Column @prop='type' />
+    <Column @prop='details'>
+      {{json-stringify data.details}}
+    </Column>
+    <Column @prop='error'>
+      {{json-stringify data.error}}
+    </Column>
+    <Column @prop='duration' />
+    <Column @prop='enqueuedAt'>
+      {{date-time-format
+        data.enqueuedAt
+        year='numeric'
+        month='numeric'
+        day='numeric'
+        hour='numeric'
+        minute='numeric'
+        second='numeric'
+      }}
+    </Column>
+    <Column @prop='startedAt'>
+      {{date-time-format
+        data.startedAt
+        year='numeric'
+        month='numeric'
+        day='numeric'
+        hour='numeric'
+        minute='numeric'
+        second='numeric'
+      }}
+    </Column>
+    <Column @prop='finishedAt'>
+      {{date-time-format
+        data.finishedAt
+        year='numeric'
+        month='numeric'
+        day='numeric'
+        hour='numeric'
+        minute='numeric'
+        second='numeric'
+      }}
+    </Column>
+  </RoyalTable>
+{{else}}
+  No results found
+{{/if}}
diff --git a/tests/unit/controllers/admin/tasks/index-test.js b/tests/unit/controllers/admin/tasks/index-test.js
new file mode 100644
index 0000000..5140259
--- /dev/null
+++ b/tests/unit/controllers/admin/tasks/index-test.js
@@ -0,0 +1,12 @@
+import { module, test } from 'qunit';
+import { setupTest } from 'meiliadmin/tests/helpers';
+
+module('Unit | Controller | admin/tasks/index', function (hooks) {
+  setupTest(hooks);
+
+  // TODO: Replace this with your real tests.
+  test('it exists', function (assert) {
+    let controller = this.owner.lookup('controller:admin/tasks/index');
+    assert.ok(controller);
+  });
+});