fixing bug with error not showing up in footer

Signed-off-by: Sean Li <[email protected]>
opensearch-project · Oct 21, 2024 · 7be5a0d · 7be5a0d
1 parent b0aa157
commit 7be5a0d
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 9 deletions.
diff --git a/...a/public/query/query_string/language_service/lib/__snapshots__/query_result.test.tsx.snap b/...a/public/query/query_string/language_service/lib/__snapshots__/query_result.test.tsx.snap
diff --git a/src/plugins/data/public/query/query_string/language_service/lib/query_result.tsx b/src/plugins/data/public/query/query_string/language_service/lib/query_result.tsx
@@ -145,9 +145,9 @@ export function QueryResult(props: { queryStatus: QueryStatus }) {
           onClick={onButtonClick}
           data-test-subj="queryResultErrorBtn"
           className="editor__footerItem"
-          flush="both"
+          color="danger"
         >
-          <EuiText size="xs" color="subdued" className="editor__footerItem">
+          <EuiText size="xs" color="danger" className="editor__footerItem">
             {i18n.translate('data.query.languageService.queryResults.error', {
               defaultMessage: `Error`,
             })}
@@ -167,7 +167,7 @@ export function QueryResult(props: { queryStatus: QueryStatus }) {
             {i18n.translate('data.query.languageService.queryResults.reasons', {
               defaultMessage: `Reasons:`,
             })}
-          </strong>
+          </strong>{' '}
           {props.queryStatus.body.error.reason}
         </EuiText>
         <EuiText size="s">
@@ -176,7 +176,7 @@ export function QueryResult(props: { queryStatus: QueryStatus }) {
               {i18n.translate('data.query.languageService.queryResults.details', {
                 defaultMessage: `Details:`,
               })}
-            </strong>
+            </strong>{' '}
             {props.queryStatus.body.error.details}
           </p>
         </EuiText>

diff --git a/src/plugins/data/public/ui/query_editor/editors/shared.tsx b/src/plugins/data/public/ui/query_editor/editors/shared.tsx
@@ -81,7 +81,7 @@ export const SingleLineInput: React.FC<SingleLineInputProps> = ({
       const blurDisposable = editor.onDidBlurEditorText(() => {
         blurTimeoutRef.current = setTimeout(() => {
           setEditorIsFocused(false);
-        }, 3000);
+        }, 500);
       });
 
       return () => {

diff --git a/src/plugins/discover/public/application/view_components/utils/use_search.ts b/src/plugins/discover/public/application/view_components/utils/use_search.ts
@@ -272,9 +272,9 @@ export const useSearch = (services: DiscoverViewServices) => {
       }
       let errorBody;
       try {
-        errorBody = JSON.parse(error.message);
+        errorBody = JSON.parse(error.body.message);
       } catch (e) {
-        errorBody = error.message;
+        errorBody = error.body.message;
       }
 
       data$.next({