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.

Sign up for GitHub

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

Jump to bottom

[UXD][AAP-25027] AWX - updates to Hosts section breadcrumbs & headers #2723

Closed
wants to merge 1 commit into from
Closed

[UXD][AAP-25027] AWX - updates to Hosts section breadcrumbs & headers #2723

wants to merge 1 commit into from

Conversation

tiyiprh
Copy link
Contributor

@tiyiprh tiyiprh commented Jul 15, 2024

https://issues.redhat.com/browse/AAP-25027

Updates to Hosts section areas breadcrumbs and headers

@github-actions github-actions bot added the AWX Label to indicate changes relevant to AWX label Jul 15, 2024
@acruzgon
Copy link
Contributor

i believe this PR can be closed in favor of #2722

@tiyiprh
Copy link
Contributor Author

tiyiprh commented Jul 17, 2024

i believe this PR can be closed in favor of #2722

@acruzgon Do I need to make the changes that I made here in #2722 as well?

@tiyiprh tiyiprh force-pushed the AWX-hosts-breadcrumb&header branch from da351fe to 8f15856 Compare July 18, 2024 22:22
@github-actions github-actions bot added the E2E End-to-end testing label Jul 18, 2024
@tiyiprh tiyiprh force-pushed the AWX-hosts-breadcrumb&header branch from 8f15856 to 2879bdd Compare August 20, 2024 16:36
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

@tiyiprh tiyiprh requested a review from daphnemaeve August 22, 2024 14:21
@tiyiprh
Copy link
Contributor Author

tiyiprh commented Aug 22, 2024

@daphnemaeve The failures don't seem related to my PR

@daphnemaeve
Copy link
Contributor

@daphnemaeve The failures don't seem related to my PR

Agreed, I'll re-run checks!

@tiyiprh
Copy link
Contributor Author

tiyiprh commented Aug 22, 2024

Closing in favor of #2722 as the changes in this PR are also in the other PR.

@tiyiprh tiyiprh closed this Aug 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nixocio nixocio Awaiting requested review from nixocio nixocio is a code owner

@jamestalton jamestalton Awaiting requested review from jamestalton jamestalton is a code owner

@akus062381 akus062381 Awaiting requested review from akus062381 akus062381 is a code owner

@vishsanghishetty vishsanghishetty Awaiting requested review from vishsanghishetty

@AlexSCorey AlexSCorey Awaiting requested review from AlexSCorey AlexSCorey is a code owner

@vidyanambiar vidyanambiar Awaiting requested review from vidyanambiar vidyanambiar is a code owner

@keithjgrant keithjgrant Awaiting requested review from keithjgrant keithjgrant is a code owner

@lgalis lgalis Awaiting requested review from lgalis lgalis is a code owner

@ZitaNemeckova ZitaNemeckova Awaiting requested review from ZitaNemeckova ZitaNemeckova is a code owner

@marshmalien marshmalien Awaiting requested review from marshmalien marshmalien is a code owner

@kialam kialam Awaiting requested review from kialam kialam is a code owner

@MilanPospisil MilanPospisil Awaiting requested review from MilanPospisil MilanPospisil is a code owner

@himdel himdel Awaiting requested review from himdel himdel is a code owner

@daphnemaeve daphnemaeve Awaiting requested review from daphnemaeve

Assignees
No one assigned
Labels
AWX Label to indicate changes relevant to AWX E2E End-to-end testing
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@tiyiprh @acruzgon @daphnemaeve