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-162668: makes tags in collection request non-case-sensitive #3226

Merged
merged 81 commits into from
Dec 3, 2024

Conversation

cmiqueo
Copy link
Contributor

@cmiqueo cmiqueo commented Nov 20, 2024

MWPW-162668: makes tags in collection call non-case-sensitive

Resolves: MWPW-162668

Before:
before

After:
after

Test URLs:

overmyheadandbody and others added 8 commits November 12, 2024 18:40
* [MWPW-160011] - GNAV light mode color changes (#3019)

* Color changes

* Promo background color fix

* Add text and subtext font color changes in megamenu

* cross cloud menu background color changes

* Fix reveiew comments

* colors hexcode formatted as lowercase for css variables

* Bug fix MWPW-161889

* MWPW-161590, 161606, 161871, 161987 [MEP] Update placeholders before updating href (#3142)

* MWPW-161590 [MEP] Update placeholders before updating href

* unit test updates

* MWPW-161871 [MEP] useblockcode action fails if page is on milo site (#3161)

* stash

* ready to publish

* interact call coming back but timing out

* working on a fast connection

* clean up ifs

* working with promise

* use camel case on let variable

* unit tests

* update unit tests

* MWPW-149504 [MILO][MEP] Move entitlements object to the same JSON file used by the library (#3047)

* create mepxlg branch

* update library

* require full hostname match

* update reference for unit test

* switch to use config instead of domain list and stub response in unit test

* update fetch to 2

* updating another fetch to 2

* restore normalizePath to use preview domains on preview links

* preload segment list json

* use getFederatedUrl instead

* import at top so we don't have to make normalizePath async

* MWPW-148129 [MILO][MEP][GNAV] Able to use select by url feature with federated link (#3064)

* add federated link function to registerInBlockActions

* add to unit test

* add unit test coverage

* initial commit

* Fix useblock code issue

---------

Co-authored-by: vgoodric <vgoodric@adobe.com>
Co-authored-by: Vivian A Goodrich <101133187+vgoodric@users.noreply.github.com>

* MWPW-161606 [MEP] Modify within modals with MEP (#3171)

initial commit publish

Co-authored-by: viloria <viloria@adobe.com>

* add conditionals

---------

Co-authored-by: Mark Perry <124626043+markpadbe@users.noreply.github.com>
Co-authored-by: viloria <viloria@adobe.com>

* [MWPW-161782] - Added logo changes for older GNAV experience (#3162)

* Added logo changes for older GNAV experience

* Lint Fix

* Logo alighned to left for older GNAV

* [MWPW-161563] CDT Overlap in Global and APAC Promotion file (#3168)

* Use schedule as source for CDT

* seperate start and end

* Added unit tests

* [MWPW-157451] [Accessibility] Merch cards | Focus is lost on comparing plans section (#3103)

* remive tabindex=0 as the attributes from merchcard div

* fresh commit with same changes as commit 1 after resolving the merge conflict

* Redesign mini-compare chart. Added checkmarks for icons. Backward compatibility using "checkmark" class. Horizontal line added.

* Revert "Redesign mini-compare chart. Added checkmarks for icons. Backward compatibility using "checkmark" class. Horizontal line added."

This reverts commit 610807a.

* run build in mas

* run build

---------

Co-authored-by: Rohit Sahu <rosahu@adobe.com>

* MWPW-159381: Replaces CSS 'body *' with '.consonant-Wrapper *' (#3152)

* Revert "MWPW-156749: Fix video CLS " (#2899) (#2900)

Revert "MWPW-156749: Fix video CLS  (#2849)"

This reverts commit d4134c8.

* [MWPW-159903] Fix quiz video marquees (#3009) (#3013)

fix quiz marquees

* [MWPW-159328] handle a case where there are not placeholders availabl… (#3014)

[MWPW-159328] handle a case where there are not placeholders available (#2998)

* [MWPW-159328] handle a case where there are not palceholders availble

* fixed typos

---------

Co-authored-by: Denys Fedotov <denlight@gmail.com>
Co-authored-by: Denys Fedotov <dfedotov@Denyss-MacBook-Pro.local>

* MWPW-146211 [MILO][MEP] Option to select all elements (#2976) (#3023)

* stash

* stash

* stash

* working well

* set updated command list for inline

* remove querySelector function

* unit test and custom block fix

* updates for in-block

* merch-card-collection unit test fixed

* unit test updates

* more unit test repair

* linting errors

* more linting

* Fix Invalid selector test

* add coverage

* force git checks to refire

* remove comment

* pass rootEl to getSelectedElements for use when needed (gnav)

* skip if clause in codecov

---------

Co-authored-by: Vivian A Goodrich <101133187+vgoodric@users.noreply.github.com>
Co-authored-by: markpadbe <markp@adobe.com>

* MWPW-158455: Promobar overlays with localnav elements in devices (#2991)

* Revert "MWPW-156749: Fix video CLS " (#2899) (#2900)

Revert "MWPW-156749: Fix video CLS  (#2849)"

This reverts commit d4134c8.

* Changing z-index of promobar and popup

* Changing z-index of promobar and popup

* Reverting z-index to 4 for promobar

---------

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>
Co-authored-by: Blaine Gunn <Blainegunn@gmail.com>
Co-authored-by: Akansha Arora <>

* Replaces CSS 'body *' with '.consonant-Wrapper *'

---------

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>
Co-authored-by: Blaine Gunn <Blainegunn@gmail.com>
Co-authored-by: Denys Fedotov <denlight@gmail.com>
Co-authored-by: Denys Fedotov <dfedotov@Denyss-MacBook-Pro.local>
Co-authored-by: Rares Munteanu <overmyheadandbody@gmail.com>
Co-authored-by: Vivian A Goodrich <101133187+vgoodric@users.noreply.github.com>
Co-authored-by: markpadbe <markp@adobe.com>
Co-authored-by: Akansha Arora <akanshaarora090@gmail.com>

* [MWPW-160948] Remove blockquote default spacing (#3159)

* [MWPW-161098] Environment-aware links support for CaaS (#3165)

* relative urls caas support

* added unit tests

* [MWPW-162059] - Fix for vertical alignment of logo on AH (#3180)

Fix for verticle alignment of logo

* Global links underlined by default (#3166)

* a links underline, a.quite added

* added styles to omit header/footer

* added default link styles for headlines and lockup area

* minor :hover style alignment on cards-horiz, marquee-anchors, brick

* removed dubs underline on links

* removed styles overrides to support double in horiz-card, marquee-anch, brick

* [MWPW-145672] Remove Gnav global link styles

* [MWPW-161624] Link farm double underline on focus

* Ensure sub text in anchors is underlined correctly

* Ensure sub text in anchors is underlined correctly

---------

Co-authored-by: thi64146 <thi64146@adobe.com>

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

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

This reverts commit f379815.

Co-authored-by: Rares Munteanu <overmyheadandbody@gmail.com>

* [MWPW-145672] Change link color only on hover (#3186)

---------

Co-authored-by: Dev Ashish Saradhana <41122199+Deva309@users.noreply.github.com>
Co-authored-by: Vivian A Goodrich <101133187+vgoodric@users.noreply.github.com>
Co-authored-by: Mark Perry <124626043+markpadbe@users.noreply.github.com>
Co-authored-by: viloria <viloria@adobe.com>
Co-authored-by: Rahul Gupta <rahulgupta999@gmail.com>
Co-authored-by: rohitsahu <sahurohit9@gmail.com>
Co-authored-by: Rohit Sahu <rosahu@adobe.com>
Co-authored-by: cmiqueo <64917520+cmiqueo@users.noreply.github.com>
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>
Co-authored-by: Blaine Gunn <Blainegunn@gmail.com>
Co-authored-by: Denys Fedotov <denlight@gmail.com>
Co-authored-by: Denys Fedotov <dfedotov@Denyss-MacBook-Pro.local>
Co-authored-by: Rares Munteanu <overmyheadandbody@gmail.com>
Co-authored-by: markpadbe <markp@adobe.com>
Co-authored-by: Akansha Arora <akanshaarora090@gmail.com>
Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>
Co-authored-by: thi64146 <thi64146@adobe.com>
Co-authored-by: Narcis Radu <github@narcisradu.ro>
Copy link
Contributor

aem-code-sync bot commented Nov 20, 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 Nov 20, 2024

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

Copy link

codecov bot commented Nov 20, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.38%. Comparing base (6f4d98c) to head (e0722c9).
Report is 17 commits behind head on stage.

Additional details and impacted files
@@           Coverage Diff           @@
##            stage    #3226   +/-   ##
=======================================
  Coverage   96.37%   96.38%           
=======================================
  Files         245      245           
  Lines       56716    56725    +9     
=======================================
+ Hits        54661    54675   +14     
+ Misses       2055     2050    -5     

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

@cmiqueo cmiqueo changed the title MWPW-162668: makes tags in collection call non-case-sensitive MWPW-162668: makes tags in collection request non-case-sensitive Nov 20, 2024
Copy link
Contributor

@robert-bogos robert-bogos left a comment

Choose a reason for hiding this comment

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

👍

Copy link
Contributor

Reminder to set the Ready for Stage label - to queue this to get merged to stage & production.

@gmirijan gmirijan added caas-configurator CaaS Configurator verified PR has been E2E tested by a reviewer Ready for Stage labels Nov 22, 2024
@milo-pr-merge milo-pr-merge bot merged commit 82dc096 into stage Dec 3, 2024
29 checks passed
@milo-pr-merge milo-pr-merge bot deleted the MWPW-162668 branch December 3, 2024 09:07
@milo-pr-merge milo-pr-merge bot mentioned this pull request Dec 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
caas-configurator CaaS Configurator Ready for Stage verified PR has been E2E tested by a reviewer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants