Skip to content

Commit

Permalink
feat(@cubejs-client/react): Adding queryError to QueryBuilder for sho…
Browse files Browse the repository at this point in the history
…wing dryRun errors (#2262)

Co-authored-by: Peter Steffey <psteffey@directdefense.com>
Co-authored-by: Alex.V <vaspiring@gmail.com>
  • Loading branch information
3 people committed Mar 5, 2021
1 parent d269a38 commit 61bac0b
Showing 1 changed file with 8 additions and 0 deletions.
8 changes: 8 additions & 0 deletions packages/cubejs-client-react/src/QueryBuilder.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -211,6 +211,8 @@ export default class QueryBuilder extends React.Component {
meta,
metaError,
query,
queryError,
orderMembers = [],
chartType,
pivotConfig,
validatedQuery,
Expand Down Expand Up @@ -269,6 +271,7 @@ export default class QueryBuilder extends React.Component {
meta,
metaError,
query,
error: queryError, // Match same name as QueryRenderer prop
validatedQuery,
isQueryPresent: this.isQueryPresent(),
chartType,
Expand Down Expand Up @@ -415,7 +418,9 @@ export default class QueryBuilder extends React.Component {
this.setState({
...finalState,
query: nextQuery,
orderMembers: currentOrderMembers,
missingMembers,
queryError: null,
});

let pivotQuery = {};
Expand Down Expand Up @@ -448,6 +453,9 @@ export default class QueryBuilder extends React.Component {
}
} catch (error) {
console.error(error);
this.setState({
queryError: error
});
}
}
}
Expand Down

0 comments on commit 61bac0b

Please sign in to comment.