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 #3331

Merged
merged 8 commits into from
Dec 10, 2024
Merged

[Release] Stage to Main #3331

merged 8 commits into from
Dec 10, 2024

Conversation

markpadbe and others added 3 commits December 9, 2024 09:18
* Initial updates to handle new UX requirements

* Add segment udpates

* Fix class

* Fix center text

* rtl updates for segments

* Remove rtl test

* Add back padding

* Fix padding and columns

* Add fix for bad class

* Remove comment

* Update tests

* Fix test

* Update libs/blocks/timeline/timeline.js

---------

Co-authored-by: Vivian A Goodrich <101133187+vgoodric@users.noreply.github.com>
* Fix: Setting allowedOrigins for standalone gnav

* Fix: Default handling
…idual SVG assets (#3259)

* updated features/icons to pull from federal and allow the icons set to be sharepoint authorable

* bettter icon-spacing handling

* preload federated.js instead of non used icons.svg on util decorateIcon()

* Updated preload federated as script type not fetch

* Minor clean up based on pr feedback

* remove console

* addressed some issues found w/ icons not rending in tables due to race condition w/ decorateIcon()

* lana instead of console

* no 100% height

* coverage

* full coverage

* minor cleanup

* minor cleanup

* preload first section icons, spread syntax

* no cons

* Performace refactor - loadIcons to hold most related functionallity. Preloaded inView icons and defered others till postSectionLoad.

* async decTooltips and refactor to not chain func calls

* fix: icon alignment in georouting modal's button

* move render blocking code to utils

* fix import of test method

* remove condition that causes error when no icon is in view

* remove redundant link load

* remove extra federated.js import

* small change to push branch

* Fed Icons - Table icon decoration and collapse icon wrapping (#3202)

* reselect icons after area is decorated

* fix inline margin issue in table collapse titles

---------

Co-authored-by: Saugat Malla <saugat@TBXTOR-2021MBP16-SaugatMalla.local>
Co-authored-by: Saugat Malla <saugat013@gmail.com>
Co-authored-by: Sartxi <sean.archibeque@gmail.com>
@milo-pr-merge
Copy link
Contributor Author

milo-pr-merge bot commented Dec 9, 2024

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

@milo-pr-merge milo-pr-merge bot requested a review from ryanmparrish as a code owner December 9, 2024 09:19
Copy link
Contributor

aem-code-sync bot commented Dec 9, 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 Dec 9, 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

codecov bot commented Dec 9, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.42%. Comparing base (b72ed94) to head (39eafc8).
Report is 10 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff            @@
##             main    #3331    +/-   ##
========================================
  Coverage   96.41%   96.42%            
========================================
  Files         246      246            
  Lines       57567    57676   +109     
========================================
+ Hits        55504    55613   +109     
  Misses       2063     2063            

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

@aem-code-sync aem-code-sync bot temporarily deployed to stage December 9, 2024 09:58 Inactive
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 by default. Use a negated ignore pattern (like "--ignore-pattern '!<relative/path/to/filename>'") 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 by default. Use a negated ignore pattern (like "--ignore-pattern '!<relative/path/to/filename>'") 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 by default. Use a negated ignore pattern (like "--ignore-pattern '!<relative/path/to/filename>'") 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 by default. Use a negated ignore pattern (like "--ignore-pattern '!<relative/path/to/filename>'") to override.

@milo-pr-merge milo-pr-merge bot requested a review from a team as a code owner December 9, 2024 12:09
updating the job to manual trigger

Co-authored-by: Santoshkumar Sharanappa Nateekar <nateekar@Santoshkumars-MacBook-Pro.local>
* MWPW-154147-spectrum switch

* MWPW-154147-spectrum switch

* MWPW-154147-spectrum switch test

* MWPW-154147-spectrum switch test

---------

Co-authored-by: Sonja Popovic <cod55010@adobe.com>
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.

update tooltip locator

Co-authored-by: Santoshkumar Sharanappa Nateekar <nateekar@SantoshumarsMBP.attlocal.net>
@milo-pr-merge milo-pr-merge bot merged commit 41ef7a9 into main Dec 10, 2024
45 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.