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

Merged
merged 7 commits into from
Dec 11, 2024
Merged

[Release] Stage to Main #3345

merged 7 commits into from
Dec 11, 2024

Conversation

swamu and others added 4 commits December 10, 2024 09:18
…c for not signed in (#3324)

* remobe personalization for signed out, change domain names for stage and prod

* dummy-pr

* remove hlx and aem check for getting url

* A&T property check on default production

* Fix decode URI at the base

* Fix amcv cookie name post decode

* Add Amcv encoding during setting cookie
* Fixing the issue that sidenav can't be collapsed.

* Unit test update.

* Move string to a const

* results announce

* trying to fix the unit test

* role status > aria-live polite

* aria-live polite test update
* Sets Auto Select for cotentType as default

* Adds beta link to original bulkpublisher

* Implements qa feedback and other imnprovements

* ESlint errors

* Implements previous PR review comments

* Fixes ESLint errors

---------

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

milo-pr-merge bot commented Dec 10, 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 review from chrischrischris and a team as code owners December 10, 2024 09:18
Copy link
Contributor

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

codecov bot commented Dec 10, 2024

Codecov Report

Attention: Patch coverage is 98.72041% with 20 lines in your changes missing coverage. Please review.

Project coverage is 96.43%. Comparing base (39eafc8) to head (4e29c5c).
Report is 10 commits behind head on main.

Files with missing lines Patch % Lines
libs/features/mas/src/log.js 92.79% 8 Missing ⚠️
libs/features/mas/src/lana.js 95.37% 5 Missing ⚠️
libs/features/mas/src/price/utilities.js 99.27% 2 Missing ⚠️
libs/martech/helpers.js 95.12% 2 Missing ⚠️
libs/blocks/merch-card/merch-card.js 0.00% 1 Missing ⚠️
libs/features/mas/src/price/template.js 99.76% 1 Missing ⚠️
libs/features/mas/src/variants/product.js 85.71% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3345      +/-   ##
==========================================
+ Coverage   96.42%   96.43%   +0.01%     
==========================================
  Files         246      253       +7     
  Lines       57676    58761    +1085     
==========================================
+ Hits        55613    56669    +1056     
- Misses       2063     2092      +29     

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

* add build artifacts

---------

Co-authored-by: Nicolas Peltier <npeltier@adobe.com>
Co-authored-by: Ilyas Türkben <tuerkben@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: Ilyas Türkben <ilyas@adobe.com>
@milo-pr-merge milo-pr-merge bot merged commit badc68d into main Dec 11, 2024
38 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.