Please review pull request #57: (#7554) Link latest report to actual report on front page opened by (devonharless)

Description:

The latest report time in the node listing on the front page was
not linking to the latest report. Added an active link to the
timestamp in order to navigate to the individual report.

  • Opened: Wed Nov 16 19:06:23 UTC 2011
  • Based on: puppetlabs:master (4d2c7b72772acb2f6f7e878796a83b3986aaeb2a)
  • Requested merge: devonharless:tickets/master/7554-latest-report-link (a45a326877ab46ed5ef397e0c2d33c96cac052b9)

Diff follows:

diff --git a/app/views/nodes/_nodes.html.haml b/app/views/nodes/_nodes.html.haml
index 62ea029..2e79035 100644
--- a/app/views/nodes/_nodes.html.haml
+++ b/app/views/nodes/_nodes.html.haml
@@ -61,7 +61,7 @@
               - else
                 = sources.map{|s| link_to(s.name,s)}.join(", ")
           %td.latest_report
-            = node.last_apply_report ? node.last_apply_report.time : "Has not reported"
+            = node.last_apply_report ? link_to(h(node.last_apply_report.time), node.last_apply_report) : "Has not reported"
           - column_filter[ selected_status ].each do |status|
             %td
               = node.last_apply_report.metric_value("resources", status).to_i if node.last_apply_report

    

--
You received this message because you are subscribed to the Google Groups "Puppet Developers" group.
To post to this group, send email to puppet-dev@googlegroups.com.
To unsubscribe from this group, send email to puppet-dev+unsubscr...@googlegroups.com.
For more options, visit this group at http://groups.google.com/group/puppet-dev?hl=en.

Reply via email to