Always provide arguments instead of models to job links

This invoked the model hook, which will load the model and redirect correctly based on permissions

Always provide arguments instead of models to job links
This invoked the model hook, which will load the model and redirect correctly based on permissions
1c0d133f · Michael Lange · 93471649 · 1c0d133f · 1c0d133f
Commit 1c0d133f authored 7 years ago by Michael Lange
Hide whitespace changes
Inline Side-by-side

Showing

with 2 additions and 2 deletions
+2 -2
--- a/ui/app/controllers/jobs/index.js
+++ b/ui/app/controllers/jobs/index.js
@@ -53,7 +53,7 @@ export default Controller.extend(Sortable, Searchable, {

  actions: {
    gotoJob(job) {
-      this.transitionToRoute('jobs.job', job);
+      this.transitionToRoute('jobs.job', job.get('plainId'));
    },

    refresh() {

--- a/ui/app/templates/components/job-row.hbs
+++ b/ui/app/templates/components/job-row.hbs
-<td data-test-job-name>{{#link-to "jobs.job" job class="is-primary"}}{{job.name}}{{/link-to}}</td>
+<td data-test-job-name>{{#link-to "jobs.job" job.plainId class="is-primary"}}{{job.name}}{{/link-to}}</td>
 <td data-test-job-status>
  <span class="tag {{job.statusClass}}">{{job.status}}</span>
 </td>