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

[Release] Stage to Main #3093

Merged
merged 9 commits into from
Oct 29, 2024
Merged

[Release] Stage to Main #3093

merged 9 commits into from
Oct 29, 2024

Conversation

rohitsahu and others added 9 commits October 28, 2024 09:17
…ns (#3088)

* Add "Save now" to the OST CTAs options

* review comment

* UT fix

* dummy commit

* Revert "dummy commit"

This reverts commit a73f1e5.

* dummy commit for intermittent test failure

* Revert "dummy commit for intermittent test failure"

This reverts commit 24c3d4b.

* dummy commit

(cherry picked from commit a73f1e5)

* Revert "dummy commit"

This reverts commit 8d2d7bc.

---------

Co-authored-by: Rohit Sahu <rosahu@adobe.com>
* Adding a dynamic nav status button into the global nav to aid content QA in understanding which nav is active

* Code review comments: cleaned css and added prod check in loadDeferred to disable status. Updated tests

* Moving styles into utils, as loading within module caused CLS

* CSS typo

* ESLint error

* Change for clarity in utils

* PR requested changes

* Changes requested from QA
…#3018)

* Fixed max width of heading and badge if badge is present for product cards

* review comment

* review comment

* run build

---------

Co-authored-by: Rohit Sahu <rosahu@adobe.com>
* add .no-track to quiz cards
* add .no-track to quiz-entry cards

Resolves: [MWPW-152331](https://jira.corp.adobe.com/browse/MWPW-152331)
* MWPW-159555 Add support for lana-sample query param

The `lana-sample` query param will override any sample rate defined elsewhere.  Setting it to "100" will force lana to send everything to splunk.

* Disable es-lint on lana.js

* Use explicit eslint labels

* Fix lana test to not intermittently fail

* Add invalid value test
* fixed style for .empty-asset:empty to still take up dom space split w/ copy

* set default lockup size for if no lockup variant gets authored

* moved block lockup styled to main-copy and adjusted row-lockup to check icon and lockup suffix

* large default

* added support for breakpoint fill button styles

* removed bg-tablet selectors in wrong media query

* added extra `con-button.button-xxl` to support blocks w/ element level classes.

* block button class target main-copy and removed so rows dont inherit

* added .no-min-height

* Test coverage for lockup size in row variant
* Graybox display specificity fix

* ensure that the graybox block will never be seen, regardless of page content or setup

Resolves: [MWPW-160469](https://jira.corp.adobe.com/browse/MWPW-160469)

* remove odd unintended z-index changes

* missed a ;
Adding tabindex to social share icons to make sure users can tab to them
@milo-pr-merge
Copy link
Contributor Author

milo-pr-merge bot commented Oct 28, 2024

Testing can start @adobecom/miq-sot @adobecom/bacom-sot @adobecom/homepage-sot @adobecom/creative-cloud-sot @adobecom/document-cloud-sot

Copy link
Contributor

aem-code-sync bot commented Oct 28, 2024

Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch and validate page speed.
In case there are problems, just click a checkbox below to rerun the respective action.

  • Re-run PSI checks
  • Re-sync branch
Commits

Copy link
Contributor

aem-code-sync bot commented Oct 28, 2024

Page Scores Audits Google
📱 /?martech=off PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
🖥️ /?martech=off PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI

Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

Copy link

codecov bot commented Oct 28, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.45%. Comparing base (c61c4d6) to head (ff054af).
Report is 76 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3093      +/-   ##
==========================================
+ Coverage   96.43%   96.45%   +0.01%     
==========================================
  Files         245      246       +1     
  Lines       55746    55909     +163     
==========================================
+ Hits        53757    53925     +168     
+ Misses       1989     1984       -5     

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

@tedAir tedAir requested review from tedAir and removed request for a team October 28, 2024 15:16
@milo-pr-merge milo-pr-merge bot merged commit fc2cb0a into main Oct 29, 2024
48 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.