You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@superset.apache.org by be...@apache.org on 2019/03/05 09:12:15 UTC

[incubator-superset] branch lyftga updated: Sparkline dates aren't formatting in Time Series Table (#6976)

This is an automated email from the ASF dual-hosted git repository.

beto pushed a commit to branch lyftga
in repository https://gitbox.apache.org/repos/asf/incubator-superset.git


The following commit(s) were added to refs/heads/lyftga by this push:
     new 1ef7fb6  Sparkline dates aren't formatting in Time Series Table (#6976)
1ef7fb6 is described below

commit 1ef7fb61785619e8e53ed60de9ea528384c740be
Author: Kim Truong <47...@users.noreply.github.com>
AuthorDate: Tue Mar 5 01:11:59 2019 -0800

    Sparkline dates aren't formatting in Time Series Table (#6976)
    
    * Exclude venv for python linter to ignore
    
    * Fix NaN error
---
 superset/assets/src/visualizations/TimeTable/TimeTable.jsx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/superset/assets/src/visualizations/TimeTable/TimeTable.jsx b/superset/assets/src/visualizations/TimeTable/TimeTable.jsx
index 38b9d22..24ebc80 100644
--- a/superset/assets/src/visualizations/TimeTable/TimeTable.jsx
+++ b/superset/assets/src/visualizations/TimeTable/TimeTable.jsx
@@ -148,7 +148,7 @@ class TimeTable extends React.PureComponent {
           renderTooltip={({ index }) => (
             <div>
               <strong>{formatNumber(column.d3format, sparkData[index])}</strong>
-              <div>{formatTime(column.dateFormat, entries[index].time)}</div>
+              <div>{formatTime(column.dateFormat, new Date(entries[index].time))}</div>
             </div>
           )}
         />