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

MWPW-138966: aa for gnav hamburger menu button #1592

Merged
merged 4 commits into from
Dec 6, 2023

Conversation

@robert-bogos robert-bogos added run-nala Run Nala Test Automation against PR needs-verification PR requires E2E testing by a reviewer labels Nov 29, 2023
@robert-bogos robert-bogos self-assigned this Nov 29, 2023
Copy link
Contributor

aem-code-sync bot commented Nov 29, 2023

Hello, I'm the AEM Code Sync Bot and I will run some test suites that validate the page speed.
In case there are problems, just click the checkbox below to rerun the respective action.

  • Re-run PSI Checks

Copy link
Contributor

aem-code-sync bot commented Nov 29, 2023

Page Scores Audits Google
/drafts/rbogos/gnav-layout?martech=off PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI

@SilviuLCF SilviuLCF self-requested a review November 29, 2023 12:33
Copy link

codecov bot commented Dec 5, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (1dc3a72) 95.54% compared to head (320cc63) 95.53%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1592      +/-   ##
==========================================
- Coverage   95.54%   95.53%   -0.02%     
==========================================
  Files         150      150              
  Lines       38702    38704       +2     
==========================================
- Hits        36979    36976       -3     
- Misses       1723     1728       +5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@narcis-radu narcis-radu assigned SilviuLCF and unassigned robert-bogos Dec 5, 2023
Copy link

@SilviuLCF SilviuLCF left a comment

Choose a reason for hiding this comment

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

PR can be merged to main:
Testing details: https://jira.corp.adobe.com/browse/MWPW-138966

@overmyheadandbody overmyheadandbody merged commit 08fafbf into adobecom:main Dec 6, 2023
11 checks passed
chrischrischris pushed a commit that referenced this pull request Dec 6, 2023
* aa for gnav hamburger menu button

* hotfix

---------

Co-authored-by: Narcis Radu <[email protected]>
yesil pushed a commit to yesil/milo that referenced this pull request Dec 7, 2023
* aa for gnav hamburger menu button

* hotfix

---------

Co-authored-by: Narcis Radu <[email protected]>
vgoodric pushed a commit to vgoodric/bootcamp-milo that referenced this pull request Feb 1, 2024
* aa for gnav hamburger menu button

* hotfix

---------

Co-authored-by: Narcis Radu <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-verification PR requires E2E testing by a reviewer run-nala Run Nala Test Automation against PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants