Skip to content

Hide the Workflow Visualizer button on Job Slices#372

Merged
cigamit merged 1 commit into
mainfrom
hide_visualizer_slices
Jun 9, 2026
Merged

Hide the Workflow Visualizer button on Job Slices#372
cigamit merged 1 commit into
mainfrom
hide_visualizer_slices

Conversation

@cigamit

@cigamit cigamit commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Job slices create an "adhoc" Workflow, so there is no actual Workflow behind the scene. The visualizer errors when clicking it on these job slices as there is no actual Workflow.

@cigamit cigamit requested a review from TheWitness June 8, 2026 21:16
@cigamit cigamit self-assigned this Jun 8, 2026
Copilot AI review requested due to automatic review settings June 8, 2026 21:16

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the Workflow Output toolbar in the AWX UI to avoid showing (and attempting to navigate with) the Workflow Visualizer action for job slices, which do not have a real backing workflow template and currently error when the visualizer is invoked.

Changes:

  • Derive the visualizer target template ID from job.summary_fields.workflow_job_template and only navigate when it exists.
  • Conditionally render the “Edit workflow” (visualizer) button only when a workflow template ID is present.
  • Extend the toolbar test coverage to assert the visualizer button is hidden when workflow_job_template is null.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
awx/ui/src/screens/Job/WorkflowOutput/WorkflowOutputToolbar.js Hides the visualizer button and guards navigation when there is no workflow template behind the job (e.g., slices).
awx/ui/src/screens/Job/WorkflowOutput/WorkflowOutputToolbar.test.js Adds coverage to ensure the visualizer button is not rendered when workflow_job_template is absent.

@cigamit cigamit merged commit 81c7ea8 into main Jun 9, 2026
1 check passed
@cigamit cigamit deleted the hide_visualizer_slices branch June 9, 2026 19:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants