From 61391dec1cf958aa8929a2484083a79279ed546c Mon Sep 17 00:00:00 2001 From: Phil Renaud Date: Tue, 16 May 2023 15:33:48 -0400 Subject: [PATCH 1/2] Treated same-route as sub-route and didnt cancel watchers --- ui/app/mixins/with-watchers.js | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/ui/app/mixins/with-watchers.js b/ui/app/mixins/with-watchers.js index bc5dc0909c6..b3af0fabe88 100644 --- a/ui/app/mixins/with-watchers.js +++ b/ui/app/mixins/with-watchers.js @@ -41,13 +41,16 @@ export default Mixin.create(WithVisibilityDetection, { actions: { willTransition(transition) { // Don't cancel watchers if transitioning into a sub-route + // Make sure, if it starts with the route name, that it's not the same route if ( !transition.intent.name || - !transition.intent.name.startsWith(this.routeName) + !( + transition.intent.name.startsWith(this.routeName) && + this.routeName !== transition.intent.name + ) ) { this.cancelAllWatchers(); } - // Bubble the action up to the application route return true; }, From c26747226d1d1e14853e9941950493377e94173f Mon Sep 17 00:00:00 2001 From: Phil Renaud Date: Wed, 17 May 2023 16:35:57 -0400 Subject: [PATCH 2/2] Changelog --- .changelog/17214.txt | 3 +++ 1 file changed, 3 insertions(+) create mode 100644 .changelog/17214.txt diff --git a/.changelog/17214.txt b/.changelog/17214.txt new file mode 100644 index 00000000000..2f2e15d77b0 --- /dev/null +++ b/.changelog/17214.txt @@ -0,0 +1,3 @@ +```release-note:bug +ui: fixes an issue where the allocations table on child (periodic, parameterized) job pages wouldn't update when accessed via their parent +```