[GitHub] rumbin commented on issue #5584: [fix] Enforcing main dttm column

2018-08-11 Thread GitBox
rumbin commented on issue #5584: [fix] Enforcing main dttm column URL: https://github.com/apache/incubator-superset/pull/5584#issuecomment-412256083 This PR would remedy #3629, I suppose. Thanks a lot! This is an

[GitHub] villebro commented on issue #5604: TableViz raise Unexpected error when table include null value

2018-08-11 Thread GitBox
villebro commented on issue #5604: TableViz raise Unexpected error when table include null value URL: https://github.com/apache/incubator-superset/issues/5604#issuecomment-412254531 @Wujiao233 This was reported on `0.26.3` but the PR is against `master`, can you verify that you are able

[GitHub] hughhhh commented on issue #5597: No exception on no data

2018-08-11 Thread GitBox
hug commented on issue #5597: No exception on no data URL: https://github.com/apache/incubator-superset/pull/5597#issuecomment-412302928  This is an automated message from the Apache Git Service. To respond to the

[GitHub] Wujiao233 commented on issue #5605: [fix] TableViz render Null Value Error

2018-08-11 Thread GitBox
Wujiao233 commented on issue #5605: [fix] TableViz render Null Value Error URL: https://github.com/apache/incubator-superset/pull/5605#issuecomment-412267412 It's already fix in current version of `master` This is an

[GitHub] Wujiao233 closed pull request #5605: [fix] TableViz render Null Value Error

2018-08-11 Thread GitBox
Wujiao233 closed pull request #5605: [fix] TableViz render Null Value Error URL: https://github.com/apache/incubator-superset/pull/5605 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a