This repository has been archived by the owner on Nov 13, 2023. It is now read-only.
[MWPW-133057] Search Marquee Keyword Tracking #909
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Hello, I'm Franklin Bot and I will run some test suites that validate the page speed.
|
|
|
JingleH
added a commit
that referenced
this pull request
Jun 23, 2023
* searchmarquee+ckgpills+categorylist+performance * nonblocking block-loading * mwpw-131491 fallback error message * performance: dynamic limitCnt on screenSizes * fix(props-scope-limit): done (#850) * [MWPW-131914] Load Better Img in background (#851) * fix(load-better): done * fix(load-better): fixed loadMoreButton bug * fix(template-list): adding back lastRow remove to fix query url * [MWPW-131912] Lazy Load templates.js (#853) * small regression fix * templates.js revamp * LCP configuration * update LCP function (#854) * dynamic load search marquee autocomplete (#855) * dynamic load search marquee autocomplete * small css fix * [MWPW-131912-4]: attempt to fix CLS opportunities (#856) * attempt to fix CLS opportunities * remove hard coded width * Template list revamp perf (#859) * misc improvements * back to use dummy image --------- Co-authored-by: Jingle Huang <[email protected]> * fixmes for now having only one template-list block * [MWPW-131458] Page restructure related fixes (#865) * refactored fetchSheetData * [MWPW-132212] Template v2 Search Feature Update (#870) * search function done for EN * reinstate sheet powered page validation * findMatchExistingSEOPage col name update * redirectSearch col name update * renaming vars * breadcrumbs initial impl * breadcrumbs desktop only * [MWPW-132220] Template-v2: Investigate CKG Default Bottom of the Pages (#875) * only append SEO linklist if replaceable pill found * added min-height to prevent list clipping * add locale referencing in sheet fetch * load templates.js after all blocks loaded * hide search-marquee on non-desktop * move templates.js out of scripts.js * added template-list tasks fallback * await heading update * 1st fallback with only tasks * [MWPW-131458] Template List CSS Tweak (#881) * CSS tweaking for new template-list * Update template-list.css * [MWPW-131458-3] Template List v2 QA round 2 (#884) * [MWPW-131458-3] Template List QA round 2 * [MWPW-131458-3] fallback text fix * tweaking template-list padding * Update template-list.css * fix desktop and mobile category list structure conflict * fix(search-marquee): shouldn't always fire search with current tasks * accessibility breadcrumbs * hide breadcrumbs for templates * suppress gnav breadcrumbs for templates (#890) * no gnav breadcrumbs for templates * use null than undefined --------- Co-authored-by: Jingle Huang <[email protected]> * remove template-list FIXME * sanitize breadcrumbs * fix await * remove sticky searchbar unpredictable JS based hover effect * hide non-en template breadcrumbs * Template list backward compatible (#896) * attemp backward compatability * memoize to prevent double fetch * adjust breadcrumbs and searchbar on old content * added ckgid metadata --------- Co-authored-by: Jingle Huang <[email protected]> * fix(template-list-v2): ckg pill fetch fix * fix(content-replacement): scope expansion * better backward compatibility * fix(template-list) simplify fetchLegacyAllTemplatesMetadata * fix(template-list): metadata fetch fallback logic fix * import fix * test * adding TODO * oldSheet templateTasks * wip * localized breadcrumbs * [MWPW-129863] Template v2 QA round 4 (#898) backward compatibility with old/new sheet column mapping * sanitize shorttitle * fix * stop expecting ratios to always be specified (#901) * Fallback check for placeholder format (#902) * stop expecting ratios to always be specified * PH ratio fallback fix * defaulting placeholder format (#903) * use ratio from metadata as fallback * use ratio from metadata as fallback * rebase to revamp branch form upstream * use getMetadata in constructprops * Update all-templates-metadata.js (#904) * object adaptation correction (#905) * add missing await (#906) * object adaptation correction * add missing await * add missing await * task sanitization (#907) * categorylist straight redirect (#900) Co-authored-by: Jingle Huang <[email protected]> * add trailing slash to breadcrumbs (#899) Co-authored-by: Jingle Huang <[email protected]> * bring back categoryList cnt * sanitize task, topic and q (#908) * [MWPW-133057] Search Marquee Keyword Tracking (#909) * fix(search-bar-tracking): initial setup. might just work. * body json format fix * fix(search-tracking): send raw timestamp * showLinkLIst default true * added template-list error handling * fix decorateLinklist * [MWPW-133097] Long Text Block - 1st Iteration (#911) * setup long txt block * feat(long-text): basic feature / responsiveness done * fix(long-text): future-proof * feat(search-tracking): adding audience and mobile support (#913) * Reduce tbt (#910) * try to improve updatePage performance * code styles --------- Co-authored-by: Jingle Huang <[email protected]> * fix regex * copy over new repo browse-by-category block * fix for create pages * f * fix searchMarquee for templates root page * fix regex --------- Co-authored-by: Jingle Huang <[email protected]> Co-authored-by: Qiyun Dai <[email protected]>
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Please always provide the JIRA issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Resolve MWPW-133057
Description:
The search marquee now has the ability to send the searched text to a log sheet(find in the bottom of this PR). As a temporary solution, we'll be doing this in a 'fire and forget' fashion. There could be delayed/missed submissions in tracking but the search feature won't be affected.
To test, fire a random search then go to the log sheet and watch the new row appear with the keyword you put in the search bar.
Test URLs:
Target sheet: https://adobe.sharepoint.com/:x:/r/sites/CC-Express/_layouts/15/Doc.aspx?sourcedoc=%7B0783B265-FE6F-49E6-AB90-A381B19B0AAA%7D&file=search-terms-log.xlsx&action=default&mobileredirect=true&cid=059e4557-d893-4d44-9d91-8cbdb931ce27