Skip to content

Commit

Permalink
ui: remove warning when auto refresh enable
Browse files Browse the repository at this point in the history
The warning being displayed about old active executions
was not bein properly removed when the auto refresh was
turned back on.
This commit fixes this for both Statements and Transactions
pages, on Active Executions.

Fixes CRDB-35837

Release note (ui change): Properly remove warning of old
date on Active Executions when auto refresh is enabled.
  • Loading branch information
maryliag committed Feb 2, 2024
1 parent 9434801 commit 2fa7faa
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -219,6 +219,7 @@ export const ActiveStatementsView: React.FC<ActiveStatementsViewProps> = ({
const onSubmitToggleAutoRefresh = () => {
// Refresh immediately when toggling auto-refresh on.
if (!isAutoRefreshEnabled) {
setDisplayRefreshAlert(false);
refreshLiveWorkload();
}
onAutoRefreshToggle(!isAutoRefreshEnabled);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -218,6 +218,7 @@ export const ActiveTransactionsView: React.FC<ActiveTransactionsViewProps> = ({
const onSubmitToggleAutoRefresh = () => {
// Refresh immediately when toggling auto-refresh on.
if (!isAutoRefreshEnabled) {
setDisplayRefreshAlert(false);
refreshLiveWorkload();
}
onAutoRefreshToggle(!isAutoRefreshEnabled);
Expand Down

0 comments on commit 2fa7faa

Please sign in to comment.