Skip to content

Commit

Permalink
cluster-ui: derive app name from route parameter in cluster-ui
Browse files Browse the repository at this point in the history
Fixes: #70998

Release justification: category 2

Previously, we were deriving the selected app name from the
query string parameter in for the statements page in the
cluster-ui package. The selected app name should be derived from
the route parameter for the statements page.

Release note (bug fix): the selected app name in the statements page
is now derived from the route parameters.
  • Loading branch information
xinhaoz committed Oct 1, 2021
1 parent 6961f28 commit cb31d8d
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ import {
ExecutionStatistics,
flattenStatementStats,
formatDate,
queryByName,
getMatchParamByName,
statementKey,
StatementStatistics,
TimestampToMoment,
Expand Down Expand Up @@ -144,7 +144,7 @@ export const selectStatements = createSelector(
return null;
}
let statements = flattenStatementStats(state.data.statements);
const app = queryByName(props.location, appAttr);
const app = getMatchParamByName(props.match, appAttr);
const isInternal = (statement: ExecutionStatistics) =>
statement.app.startsWith(state.data.internal_app_name_prefix);

Expand Down

0 comments on commit cb31d8d

Please sign in to comment.