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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

[Release] Stage to Main #3500

wants to merge 3 commits into from

Conversation

sharmrj and others added 2 commits January 16, 2025 09:30
* check for new nav when disabling ios scroll

* shortened the check from the previous commit
* Revert "MWPW-140452 - Icon authoring in milo using the federal repo a… (#3357)

Revert "MWPW-140452 - Icon authoring in milo using the federal repo and individual SVG assets (#3259)"

This reverts commit 81a5770.

* MWPW-165523: Fixes BETA Caas bulkpublisher auto-detect

* Minor css fix

---------

Co-authored-by: milo-pr-merge[bot] <169241390+milo-pr-merge[bot]@users.noreply.github.com>
Co-authored-by: Okan Sahin <39759830+mokimo@users.noreply.github.com>
@milo-pr-merge
Copy link
Contributor Author

milo-pr-merge bot commented Jan 16, 2025

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

Copy link
Contributor

aem-code-sync bot commented Jan 16, 2025

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 Jan 16, 2025

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 Jan 16, 2025

Codecov Report

Attention: Patch coverage is 96.75516% with 11 lines in your changes missing coverage. Please review.

Project coverage is 96.47%. Comparing base (b1351a6) to head (6eb0401).
Report is 12 commits behind head on main.

Files with missing lines Patch % Lines
libs/features/mas/src/checkout-mixin.js 97.10% 7 Missing ⚠️
libs/features/mas/src/checkout-button.js 92.30% 3 Missing ⚠️
libs/features/mas/src/variants/ccd-suggested.js 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3500      +/-   ##
==========================================
- Coverage   96.48%   96.47%   -0.02%     
==========================================
  Files         256      258       +2     
  Lines       60101    60150      +49     
==========================================
+ Hits        57989    58029      +40     
- Misses       2112     2121       +9     

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

* MWPW-162385 add wcsApiKey attribute

* MWPW-161355: Update MAS documentation (#3135)

* MWPW-159374: Update hydrate logic

to support all cta styles
organise code and code coverage

Update tests

* add workaround for ccd-suggested cards

* remove source maps

* improve code coverage

* merge MWPW-159374

* MWPW-161355: Update MAS documentation

* Example with contextual menu
* Checkout click event with correct event target
 when the CTA contains a text wrapped by a span

* cleanup all attributes during card hydration

* remove deps/mas/mas.js

* update test

* update doc

* allow list libs/features/mas/mas/dist/mas.js

* change order in ignore

* trying with wildcard

* allow list dist

* update hlxignore rules

* update hlxignore rules

* MWPW-161176: restructure ccd gallery (#3169)

restructure ccd gallery

* fix mas path

* fix undefined error in merch-icon

* fix regression in ctas size

* update doc

* fix css with checkout-link

* fix random rtl issue

* revert removal of mas.js from deps

* update ccd gallery

---------

Co-authored-by: Mariia Lukianets <lukianet@adobe.com>

* MWPW-161845: basic analytics on mas cards (#3206)

* MWPW-161845: add analytics

* fix review changes

* add build files

* MWPW-159191: MAS Freyja support (#3209)

* initial freyja commit

* increase codecov

* trival

* add qa support to ccd page

* improve page performance

* update ccd.html

* MWPW-161804: Merch card style auditing (#3216)

Refactor merch-card styles to improve structure and specificity

* Added space after price recurrence label for display-per-unit prices (#3196)

* Added space after price recurrence label for display-per-unit prices

* more specific selector

* built

* fix cr

* built

* adding example

* MWPW-162933: merge mas modules (#3248)

* MWPW-162933: merge mas modules

+ consonant-templates module from tacocat.js
into a single module

* MWPW-160755 - add tests for CCD cards (#3256)

* MWPW-160755 - add tests for CCD cards

* refactor and prepare dark mode checks

* add tests for slice cards

* activate dark for suggested and eslint fixes

* add check for cta variant

* fix multi mnemonic check

* fix lint

* add analytics check

* remove comment

* MWPW-163041: analytics fix (#3257)

* MWPW-163041: fix analytics & add docs

* add tests for analytics

* add analytics docu

* fix test

* Fix overriding of border style on merch cards (#3278)

* Fix overriding of border style on merch cards

* width change

* built

---------

Co-authored-by: Mariia Lukianets <mariia.lukianets@gmail.com>

* Bump timeout from 2 to 10 sec (#3282)

* bump to 10 seconds

* new change

* MWPW-163479: Switch to Spectrum CSS from SWC

CCD app already provides Spectrum CCS styles.

* update doc style

* update doc style

* update doc style

* update doc style

* update doc style

* update doc style

* Update styles

* update docs

* Keep SWC logic as an option

* MWPW-161645: lana logging for CCD (#3271)

* MWPW-161645: lana logging for CCD

with <mas-commerce-service data-lana-tags="consumer=ccd" ...>

lana will be enabled.

TODO:

Update docs

* Update tests

* fix review comments

* add host env attribute, fix lana override

* fix docs

* fix unit tests

* start loggin missing osi's

* fix message

* fix message

* Update libs/features/mas/src/lana.js

Co-authored-by: Axel Cureno Basurto <axelcureno@gmail.com>

* PR review changes

* avoid duplicate logging, log wcs url, limit page length in log

* fix formatString error

* fix render

* add tests

* fix loggin

* revert ccd comments

* fix milo tags

---------

Co-authored-by: Mariia Lukianets <mariia.lukianets@gmail.com>
Co-authored-by: Axel Cureno Basurto <axelcureno@gmail.com>

* fix gaps with Spectrum CSS support

* update version

* fix gaps reported by ccd teamm

* address gaps

* address gaps

* address gaps

* address gaps

* address gaps

* fix for MWPW-163718

addressed design gaps.

* addressed design gaps.

* bump the version

* add build artifacts

* Update colors based on CCD app

* Update colors based on CCD app

* build without source maps

* update Nala tests

* fix lint

* reset box-sizing for merch-card elements

* Force box-sizing on merch-card

* update styles: box-sizing

* fix ccd gallery

* update version

* limit ccd price style to their variants

* limit ccd price style to suggest variant

* preserve white spaces in prices section

* fix spacing in price

* Update version

* Update doc

* update cards

* update gallery styles

* make the gallery responsive

* align slice card ctas

* cards resize responsively

* fix 1x of gap issue

* fix description text alignement

* fix gaps in minimal widths

* fix the gallery layout

* Fix strikethrough price color in promos

* Update nala tests

* MWPW-164177: clean up card style on fragment

* Added a new section to ccd gallery

* update mas version

* fix missing default spectrum css button style

* WIP

* fix tabindex

* fix nala tests

* MWPW-164484: fix see terms focus indicator

* MWPW-164484: fix see terms focus indicator

* increase codecov

* Update doc

* Update doc

* rename constant

* Restrict updateLinks to spectrum css

* revert env related doc changes

* revert merch modal change

* revert merch modal change

* add data-href attribute to checkout-button element

* Update libs/features/mas/docs/src/mas.js.md

Co-authored-by: Nicolas Peltier <1032754+npeltier@users.noreply.github.com>

* fix docs

* Fix style regressions

* remove broken link

* update doc

* fix styles

---------

Co-authored-by: Nicolas Peltier <npeltier@adobe.com>
Co-authored-by: Mariia Lukianets <lukianet@adobe.com>
Co-authored-by: Axel Cureno Basurto <axelcureno@gmail.com>
Co-authored-by: Angelo Statescu <angelostatescu.adobe@gmail.com>
Co-authored-by: Milica Micic <cod23684@adobe.com>
Co-authored-by: Mariia Lukianets <mariia.lukianets@gmail.com>
Co-authored-by: Nicolas Peltier <1032754+npeltier@users.noreply.github.com>
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

eslint

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

https://github.com/adobecom/milo/blob/6eb040149ae852542b360bccf5fa343b3f309fee/libs/features/mas/src/hydrate.js


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

https://github.com/adobecom/milo/blob/6eb040149ae852542b360bccf5fa343b3f309fee/libs/features/mas/src/utils.js


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

https://github.com/adobecom/milo/blob/6eb040149ae852542b360bccf5fa343b3f309fee/libs/features/mas/src/variants/ccd-slice.js


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

https://github.com/adobecom/milo/blob/6eb040149ae852542b360bccf5fa343b3f309fee/libs/features/mas/src/variants/ccd-suggested.js


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

https://github.com/adobecom/milo/blob/6eb040149ae852542b360bccf5fa343b3f309fee/libs/features/mas/test/checkout-button.test.js


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

https://github.com/adobecom/milo/blob/6eb040149ae852542b360bccf5fa343b3f309fee/libs/features/mas/test/checkout-link.test.js


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

https://github.com/adobecom/milo/blob/6eb040149ae852542b360bccf5fa343b3f309fee/libs/features/mas/test/hydrate.test.js

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

@Roycethan Roycethan left a comment

Choose a reason for hiding this comment

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

Observed an issue in Stage on product catalog page where sorting button is broken and not functioning in Safari / IOS - tracked here : https://jira.corp.adobe.com/browse/MWPW-165885
Blocking this for now untill root cause is found out

@npeltier @yesil @3ch023 Can you plz check whats causing this and if its not from these changes then we can proceed with merge thank you

@Roycethan
Copy link

Observed an issue in Stage on product catalog page where sorting button is broken and not functioning in Safari / IOS - tracked here : https://jira.corp.adobe.com/browse/MWPW-165885 Blocking this for now untill root cause is found out

@npeltier @yesil @3ch023 Can you plz check whats causing this and if its not from these changes then we can proceed with merge thank you

The issue is reproducible on https://main--cc--adobecom.hlx.page/products/catalog or https://stage--cc--adobecom.hlx.page/products/catalog even without milolibs=stage , hence for now approving this PR and the issue anyway needs to be investigated to find the root cause of what's causing this.

@Roycethan Roycethan self-requested a review January 19, 2025 12:33
@mokimo mokimo deployed to milo_pr_merge January 20, 2025 01:08 — with GitHub Actions Active
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.

8 participants