Bug #20393: Foreman tasks table should be using index method instead of bulk_search - Katello - Foreman

Actions

Copy link

Bug #20393

closed

Foreman tasks table should be using index method instead of bulk_search

Added by Walden Raines over 7 years ago. Updated over 6 years ago.

Status:

Closed

Priority:

Normal

Assignee:

Dan Seethaler

Category:

Web UI

Target version:

Katello 3.4.6

Difficulty:

Triaged:

Bugzilla link:

Pull request:

https://github.com/Katello/katello/pull/6950, https://github.com/Katello/katello/pull/6949

Fixed in Releases:

Found in Releases:

Red Hat JIRA:

Description

Currently we are using the foreman tasks bulk search api in katello for our tasks tables. We should move to using the index method instead as it supports pagination.

Files

katello_3.4.2_product_pager.png

View katello_3.4.2_product_pager.png

2.3 KB

Lachlan Simpson, 06/25/2017 11:06 PM

Related issues 1 (0 open — 1 closed)

Actions

Copy link

Updated by Eric Helms over 7 years ago

Translation missing: en.field_release changed from 281 to 286

Actions

Copy link

Updated by Walden Raines about 7 years ago

Category set to Web UI
Assignee changed from Walden Raines to Dan Seethaler
Target version set to 160

Here is some more information. The katello tasks page appears on many pages in the application. A couple of examples are Content Hosts -> tasks and Products -> tasks.

Basically we need to change this:

https://github.com/katello/katello/blob/master/engines/bastion_katello/app/assets/javascripts/bastion_katello/tasks/task.factory.js#L33

To use the index method here:

https://github.com/theforeman/foreman-tasks/blob/master/app/controllers/foreman_tasks/api/tasks_controller.rb#L110

We should be able to accomplish that by using "queryPaged" from the bastion resource:

https://github.com/katello/bastion/blob/master/app/assets/javascripts/bastion/utils/bastion-resource.factory.js#L16

Actions

Copy link