Skip to content
This repository has been archived by the owner on Nov 13, 2023. It is now read-only.

Limit Autocomplete Only to EFGJ #950

Closed
wants to merge 340 commits into from
Closed

Conversation

JingleH
Copy link
Collaborator

@JingleH JingleH commented Aug 3, 2023

This is to sync with our current live PROD so that autocomplete API is not called on non-efgj locales. You should see no network request when you type in the search bar in the after link.

No ticket was created

Test URLs:

qiyundai and others added 27 commits August 1, 2023 21:55
E2E QA Fixes: Template X + CKG Linklist + CTA Carousel
MWPW-134395 | Updating icons
[MWPW-134396] Feature Grid Bug Fix
* Spacing updates

* Separated padding for highlighted texts

* Limited size of highlighted text
* update tracking params for CTA Carousel

* leave some breathing room for toggle scroll

* limit to search link

* refactor
* content-search undo replacing space

---------

Co-authored-by: Jingle Huang <[email protected]>
# Conflicts:
#	express/blocks/puf/puf.css
#	express/blocks/template-x/template-search-api-v3.js
#	express/blocks/template-x/template-x.css
#	express/blocks/template-x/template-x.js
#	express/icons/organizer.svg
#	express/icons/presentation.svg
#	express/icons/report.svg
#	express/icons/worksheet.svg
#	express/scripts/content-replace.js
#	express/scripts/scripts.js
Fix Rendering after Redirects
@aem-code-sync
Copy link

aem-code-sync bot commented Aug 3, 2023

Hello, I'm Franklin Bot and I will run some test suites that validate the page speed.
In case there are problems, just click the checkbox below to rerun the respective action.

  • Re-run PSI Checks

@JingleH JingleH closed this Aug 3, 2023
@JingleH JingleH deleted the ac-only-to-efgj branch August 3, 2023 21:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants