Merge pull request #13 from tmooney/add_dates_to_status_tables

Add Dates to Status Tables

app/controllers/models_controller.rb

@@ -7,7 +7,7 @@ def overview
 
     models_by_type = @models.group_by(&:subclass_name)
     statuses = ModelStatusQuery.new( @models.pluck(:genome_model_id) ).execute
-    @table_items = @models.page(params[:page])
+    @table_items = @models.order(:creation_date).page(params[:page])
 
     model_status_hash = @table_items.each_with_object({}) do |item, hash|
       hash[item.id] = { model: item }
@@ -26,7 +26,7 @@ def overview
 
   def status
     @model = Model.with_statuses_scope.where(genome_model_id: params[:id]).first!
-    builds = @model.builds
+    builds = @model.builds.order(:created_at)
 
     @analysis_project = @model.analysis_projects.first
 

app/presenters/build_status_table.rb

@@ -9,7 +9,7 @@ def initialize(builds, view_context)
   private
   def get_table_items(builds)
     builds.map do |build|
-      StatusTableItem.new(build.id, build.model.name, build.status, build_status_path(id: build.build_id))
+      StatusTableItem.new(build.id, build.model.name, build.created_at, build.status, build_status_path(id: build.build_id))
     end
   end
 end

app/presenters/model_status_table.rb

@@ -9,7 +9,7 @@ def initialize(models_with_status, view_context)
   private
   def get_table_items(models_with_status)
     models_with_status.map do |id, data|
-      StatusTableItem.new(id, data[:model].name, data[:status], model_status_path(id: id))
+      StatusTableItem.new(id, data[:model].name, data[:model].creation_date, data[:status], model_status_path(id: id))
     end
   end
 end

app/presenters/status_table.rb

@@ -8,5 +8,5 @@ def initialize(table_name, items)
   end
 end
 
-class StatusTableItem < Struct.new(:id, :name, :status, :uri)
+class StatusTableItem < Struct.new(:id, :name, :date, :status, :uri)
 end

app/views/shared/_status_table.html.haml

@@ -2,13 +2,16 @@
   %thead
     %tr
       %th ID
+      %th Date
       %th Status
       %th Name
   %tbody
   - table.items.each do |item|
     %tr
       %td
         = link_to(item.id, item.uri)
+      %td
+        = item.date
       %td.text-center
         = item.status
       %td