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

fix: (CXSPA-6857) - Footer tabbing fix #18777

Merged
merged 8 commits into from May 15, 2024
Merged

fix: (CXSPA-6857) - Footer tabbing fix #18777

merged 8 commits into from May 15, 2024

Conversation

Pio-Bar
Copy link
Contributor

@Pio-Bar Pio-Bar commented Apr 26, 2024

Ticket: CXSPA-6857

I have moved this assertion to a method as it was getting quite lengthy.
While testing I have noticed that toggling the node with the enter key would not enable the keyboard controls properly. This PR should fix that too.

@Pio-Bar Pio-Bar requested a review from a team as a code owner April 26, 2024 15:38
@github-actions github-actions bot marked this pull request as draft April 26, 2024 15:39
@Pio-Bar Pio-Bar marked this pull request as ready for review April 26, 2024 15:40
Copy link

cypress bot commented Apr 26, 2024

3 flaky tests on run #43627 ↗︎

0 119 2 0 Flakiness 3

Details:

Merge 1aa8240 into d5553f7...
Project: spartacus Commit: 75cc6986b4 ℹ️
Status: Passed Duration: 04:05 💡
Started: May 15, 2024 9:16 AM Ended: May 15, 2024 9:20 AM
Flakiness  cypress/e2e/ssr/pages.core-e2e.cy.ts • 3 flaky tests • SSR

View Output Video

Test Artifacts
SSR > should render homepage Test Replay Screenshots Video
SSR > should render PLP Test Replay Screenshots Video
SSR > should render PDP Test Replay Screenshots Video

Review all test suite changes for PR #18777 ↗︎

@github-actions github-actions bot marked this pull request as draft May 13, 2024 12:12
@Pio-Bar Pio-Bar marked this pull request as ready for review May 13, 2024 12:15
@github-actions github-actions bot marked this pull request as draft May 13, 2024 12:28
@Pio-Bar Pio-Bar marked this pull request as ready for review May 14, 2024 07:46
@github-actions github-actions bot marked this pull request as draft May 14, 2024 08:05
@Pio-Bar Pio-Bar marked this pull request as ready for review May 14, 2024 08:07
@github-actions github-actions bot marked this pull request as draft May 14, 2024 09:47
@Pio-Bar Pio-Bar marked this pull request as ready for review May 14, 2024 09:48
@github-actions github-actions bot marked this pull request as draft May 14, 2024 10:07
@Pio-Bar Pio-Bar marked this pull request as ready for review May 15, 2024 08:37
@github-actions github-actions bot marked this pull request as draft May 15, 2024 08:37
@Pio-Bar Pio-Bar marked this pull request as ready for review May 15, 2024 09:01
@github-actions github-actions bot marked this pull request as draft May 15, 2024 09:02
@Pio-Bar Pio-Bar marked this pull request as ready for review May 15, 2024 09:02
@github-actions github-actions bot marked this pull request as draft May 15, 2024 09:02
@Pio-Bar Pio-Bar marked this pull request as ready for review May 15, 2024 09:07
@Pio-Bar Pio-Bar merged commit 4c58e70 into develop May 15, 2024
26 checks passed
@Pio-Bar Pio-Bar deleted the bugfix/CXSPA-6857 branch May 15, 2024 09:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants