Skip to content

Commit

Permalink
Prevent debounceFetchNodes from jumping node view (#2560)
Browse files Browse the repository at this point in the history
Any of the events which trigger debounceFetchNodes should call
fetchNodes[Only] with the current props node specified. This prevents a
pipeline view from jumping to the first still-running parallel stage
when any non-first stage is focused and a handled non-pipeline_step
event is received.

Removed unnecessary fetchNodesOnly parameter. Addition for CI.
  • Loading branch information
werkt authored Jun 6, 2024
1 parent 1ce0c18 commit 6d6e0be
Showing 1 changed file with 3 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -50,9 +50,10 @@ export default class Pipeline extends Component {
this.debounceFetchNodes = debounce((karaokeOut) => {
logger.debug('sse fetch it', this.karaoke);
if (karaokeOut) {
this.pager.fetchNodesOnly({});
this.pager.fetchNodesOnly();
} else {
this.pager.fetchNodes({});
const { node } = this.props.params;
this.pager.fetchNodes({ node });
}
}, 200);
}
Expand Down

0 comments on commit 6d6e0be

Please sign in to comment.