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

New job invocation details page #1515

Merged

Conversation

pnovotny
Copy link
Contributor

New view and navigation was added for the first part of the new job invocation details page.

This feature is still an experimantal feature/tech preview and it is not yet fully completed.
Other content will be added in later phases.

@pnovotny pnovotny self-assigned this Aug 20, 2024
@pnovotny pnovotny requested review from pondrejk, lhellebr and a team August 20, 2024 01:19
@pnovotny pnovotny added the No-CherryPick PR doesnt need CherryPick to previous branches label Aug 20, 2024
@pnovotny pnovotny force-pushed the new-job-invocation-details-page branch 2 times, most recently from 0561998 to b470377 Compare August 20, 2024 09:38
@pnovotny pnovotny force-pushed the new-job-invocation-details-page branch from b470377 to b5981eb Compare August 20, 2024 12:37
Copy link
Contributor

@lhellebr lhellebr left a comment

Choose a reason for hiding this comment

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

ACK once PRT passes repeatedly (to avoid race conditions)

@pnovotny pnovotny force-pushed the new-job-invocation-details-page branch from b5981eb to 3392b0e Compare August 20, 2024 13:25
New view and navigation was added for the first part of the new job invocation details page.

This is part 1, as the new page is still not fully complete and other content will be added.
@pnovotny pnovotny force-pushed the new-job-invocation-details-page branch from 3392b0e to a51f817 Compare August 20, 2024 14:26
@pnovotny
Copy link
Contributor Author

I am trying to solve a problem, where the new details page doesn't load completely when running the PRT in our Selenium Grid w/ Chrome.
The page is only partially loaded like this:
job-inv-not-fully-loaded

But it loads fine when running the test locally w/ Firefox or Chrome:
job-inv-fully-loaded

@pnovotny
Copy link
Contributor Author

PRT on SatelliteQE/robottelo#15980 passed (twice).
@pondrejk @lhellebr Can we pls merge?

Copy link
Contributor

@LadislavVasina1 LadislavVasina1 left a comment

Choose a reason for hiding this comment

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

ACK

@pondrejk pondrejk merged commit fe9ce9e into SatelliteQE:master Aug 26, 2024
7 checks passed
@pnovotny pnovotny added CherryPick PR needs CherryPick to previous branches AutoMerge_Cherry_Picked Automatically merge the PR is PRT and all checks are passing 6.16.z and removed No-CherryPick PR doesnt need CherryPick to previous branches labels Aug 27, 2024
@pnovotny pnovotny deleted the new-job-invocation-details-page branch August 27, 2024 18:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.16.z AutoMerge_Cherry_Picked Automatically merge the PR is PRT and all checks are passing CherryPick PR needs CherryPick to previous branches
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants