Project

General

Profile

Bug #26113

Smart proxy log times not displayed.

Added by Alex Fisher 29 days ago. Updated 24 days ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Web Interface
Target version:
Difficulty:
Triaged:
Yes
Bugzilla link:
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

Foreman and smart-proxy version: 1.21.0-RC5

Log entries are not displaying timestamps.
Screenshot

missing-times.png View missing-times.png 98.5 KB Screenshot Alex Fisher, 02/22/2019 11:12 AM
Missing times

Related issues

Related to Foreman - Feature #19047: Unify the way how we display datesClosed2017-03-28

Associated revisions

Revision 8da0a865 (diff)
Added by Tomer Brisker 24 days ago

Fixes #26113 - Show timestamp in proxy log pane

When unifing the date time columns, the datatables initializer for the
logs table was changes to render the time column as it is. This does not
work properly with the new react date component, causing the column to
be empty. This commit removes the unneeded render function for this
column and corrects the helper to also display seconds for the
timestamps.

History

#1 Updated by Ewoud Kohl van Wijngaarden 28 days ago

  • Category set to Web Interface
  • Found in Releases 1.21.0 added

#2 Updated by Ewoud Kohl van Wijngaarden 25 days ago

  • Triaged changed from No to Yes

#3 Updated by Tomer Brisker 25 days ago

Digging in to this, looks like the datatable is initialized before react renders the date component, causing the component to be rendered off-screen for some reason. I'm continuing to investigate.

#4 Updated by Tomer Brisker 25 days ago

#5 Updated by The Foreman Bot 25 days ago

  • Assignee set to Tomer Brisker
  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/6511 added

#6 Updated by Ohad Levy 24 days ago

  • Target version set to 1.22.0
  • Fixed in Releases 1.22.0 added

#7 Updated by Tomer Brisker 24 days ago

  • Fixed in Releases 1.21.0 added
  • Fixed in Releases deleted (1.22.0)

#8 Updated by Tomer Brisker 24 days ago

  • Status changed from Ready For Testing to Closed

Also available in: Atom PDF