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

[UXD][AAP-25027] AWX - updates to Teams section breadcrumbs & headers #2732

Merged
merged 3 commits into from
Aug 21, 2024

Conversation

tiyiprh
Copy link
Contributor

@tiyiprh tiyiprh commented Jul 16, 2024

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

Updates to Teams section breadcrumbs & headers section breadcrumbs & headers

@github-actions github-actions bot added AWX Label to indicate changes relevant to AWX E2E End-to-end testing labels Jul 16, 2024
@tiyiprh tiyiprh force-pushed the AWX-team-breadcrumb&header branch from 854519e to b08c49c Compare July 18, 2024 14:44
@tiyiprh tiyiprh force-pushed the AWX-team-breadcrumb&header branch 2 times, most recently from e5a98fa to 9da2317 Compare August 20, 2024 13:23
@tiyiprh tiyiprh requested a review from daphnemaeve August 20, 2024 13:58
@tiyiprh tiyiprh force-pushed the AWX-team-breadcrumb&header branch from 9da2317 to 203d03e Compare August 21, 2024 13:25
@github-actions github-actions bot removed the E2E End-to-end testing label Aug 21, 2024
Copy link
Contributor

@daphnemaeve daphnemaeve left a comment

Choose a reason for hiding this comment

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

Looks great! Two last tests seem to be failing when attempting to verify the page title when editing a team, as it is expecting the old title of "Edit Team". Editing the two instances of cy.verifyPageTitle('Edit Team'); in cypress/e2e/awx/access/teams/teams.cy.ts to match the new page title should resolve that!

@github-actions github-actions bot added the E2E End-to-end testing label Aug 21, 2024
Copy link

@tiyiprh tiyiprh merged commit 3821791 into main Aug 21, 2024
42 checks passed
@tiyiprh tiyiprh deleted the AWX-team-breadcrumb&header branch August 21, 2024 18:17
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 E2E End-to-end testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants