Просмотр исходного кода

QueryInspector: Now shows error responses (#16514)

While converting this to React we missed subscribing to errors

Fixed #16512
Torkel Ödegaard 6 лет назад
Родитель
Сommit
e44d049a8e
1 измененных файлов с 11 добавлено и 1 удалено
  1. 11 1
      public/app/features/dashboard/panel_editor/QueryInspector.tsx

+ 11 - 1
public/app/features/dashboard/panel_editor/QueryInspector.tsx

@@ -39,14 +39,20 @@ export class QueryInspector extends PureComponent<Props, State> {
 
   componentDidMount() {
     const { panel } = this.props;
-    panel.events.on('refresh', this.onPanelRefresh);
+
     appEvents.on('ds-request-response', this.onDataSourceResponse);
+    appEvents.on('ds-request-error', this.onRequestError);
+
+    panel.events.on('refresh', this.onPanelRefresh);
     panel.refresh();
   }
 
   componentWillUnmount() {
     const { panel } = this.props;
+
     appEvents.off('ds-request-response', this.onDataSourceResponse);
+    appEvents.on('ds-request-error', this.onRequestError);
+
     panel.events.off('refresh', this.onPanelRefresh);
   }
 
@@ -73,6 +79,10 @@ export class QueryInspector extends PureComponent<Props, State> {
     }));
   };
 
+  onRequestError = (err: any) => {
+    this.onDataSourceResponse(err);
+  };
+
   onDataSourceResponse = (response: any = {}) => {
     if (this.state.isMocking) {
       this.handleMocking(response);