Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jobs output expandable section #2231

Merged
merged 8 commits into from May 13, 2024

Conversation

MilanPospisil
Copy link
Contributor

@MilanPospisil MilanPospisil commented May 3, 2024

https://issues.redhat.com/browse/AAP-14459
https://issues.redhat.com/browse/AAP-14717

Find job with type Playbook run. Then detail and outputs.

What is added:
Topmost expand all expands Playbooks and Tasks outputs.
When section is expanded, it renders (...) rectangle under line number.

image

@github-actions github-actions bot added the AWX Label to indicate changes relevant to AWX label May 3, 2024
@MilanPospisil MilanPospisil marked this pull request as ready for review May 3, 2024 11:42
@MilanPospisil MilanPospisil requested a review from prat98 May 6, 2024 07:15
Copy link
Member

@keithjgrant keithjgrant left a comment

Choose a reason for hiding this comment

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

This is great! Just a couple things I would suggest improving:

If you collapse all, then expand the Play, the "collapse all" button is still in the collapsed state and clicking it once doesn't expand/collapse anything. I think you might need to setCollapsedAll(false) any time a single item is expanded or collapsed.

frontend/awx/views/jobs/JobOutput/JobEventEllipsis.tsx Outdated Show resolved Hide resolved
@MilanPospisil MilanPospisil force-pushed the milan_controller_jobs_output_expandable_section branch from 1fcad09 to f4a0146 Compare May 10, 2024 09:56
@MilanPospisil MilanPospisil merged commit 3b9e9db into main May 13, 2024
17 checks passed
@MilanPospisil MilanPospisil deleted the milan_controller_jobs_output_expandable_section branch May 13, 2024 09:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AWX Label to indicate changes relevant to AWX
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants