From 184441a1bcf26ff10078e01d9c4bc45499dacd3f Mon Sep 17 00:00:00 2001
From: Vera Liu <vera.liu@airbnb.com>
Date: Fri, 18 Nov 2016 11:27:16 -0800
Subject: [PATCH] Use Alert for visualization error

---
 .../explorev2/components/ChartContainer.jsx   | 28 ++++---------------
 1 file changed, 6 insertions(+), 22 deletions(-)

diff --git a/superset/assets/javascripts/explorev2/components/ChartContainer.jsx b/superset/assets/javascripts/explorev2/components/ChartContainer.jsx
index 3ede6dbaaf32..5ae9a6574d6d 100644
--- a/superset/assets/javascripts/explorev2/components/ChartContainer.jsx
+++ b/superset/assets/javascripts/explorev2/components/ChartContainer.jsx
@@ -108,28 +108,12 @@ class ChartContainer extends React.Component {
         // finished rendering callback
       },
 
-      error(msg, xhr) {
-        let errorMsg = msg;
-        let errHtml = '';
-        try {
-          const o = JSON.parse(msg);
-          if (o.error) {
-            errorMsg = o.error;
-          }
-        } catch (e) {
-          // pass
-        }
-        errHtml = `<div class="alert alert-danger">${errorMsg}</div>`;
-        if (xhr) {
-          const extendedMsg = this.getErrorMsg(xhr);
-          if (extendedMsg) {
-            errHtml += `<div class="alert alert-danger">${extendedMsg}</div>`;
-          }
-        }
-        $(this.state.selector).html(errHtml);
-        this.render();
-        $('span.query').removeClass('disabled');
-        $('.query-and-save button').removeAttr('disabled');
+      clearError: () => {
+        this.props.actions.removeChartAlert();
+      },
+
+      error(msg) {
+        this.props.actions.chartUpdateFailed(msg);
       },
 
       d3format: (col, number) => {