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

OPT-26580 DC0994_US | Frictionless | Add sticky banner Upload CTA #2298

Merged
merged 16 commits into from
May 16, 2024

Conversation

ivanvatadobe
Copy link
Contributor

@ivanvatadobe ivanvatadobe commented May 13, 2024

POC for addition to sticky banner on frictionless pages. Mimicking behavior of a button that can interact and start the widget. Using javascript function to achieve the simulated click, but is not the plan for the final feature. The final feature will likely be built by the eng team that builds the frictionless widgets.

Resolves: OPT-26580

Test URLs for psi check:

Test URLs to view feature:

Note:

  • This ticket marked high priority because it is a blocker for test launch.
  • The branch name is dc0944stickybanner--- this was a typo when the branch was created. The test this code is being written for is called 'DC0994'. I have updated the urls above with the proper naming conventions.

Copy link
Contributor

aem-code-sync bot commented May 13, 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 May 13, 2024

Page Scores Audits Google
/?martech=off PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI

@ivanvatadobe ivanvatadobe added zero-impact verified PR has been E2E tested by a reviewer high priority Why is this a high priority? Blocker? Critical? Dependency? labels May 13, 2024
Copy link

codecov bot commented May 13, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.63%. Comparing base (9ce9b30) to head (6b50561).

Additional details and impacted files
@@            Coverage Diff             @@
##            stage    #2298      +/-   ##
==========================================
- Coverage   95.66%   95.63%   -0.03%     
==========================================
  Files         173      173              
  Lines       45018    45018              
==========================================
- Hits        43066    43055      -11     
- Misses       1952     1963      +11     

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

@aem-code-sync aem-code-sync bot temporarily deployed to dc0944stickybanner May 13, 2024 20:40 Inactive
@ivanvatadobe ivanvatadobe added the run-nala Run Nala Test Automation against PR label May 13, 2024
@vgoodric
Copy link
Contributor

For CSS, switch to using .aside.frictionless instead of .section.hide-on-step-start unless needed.

Copy link
Contributor

@vgoodric vgoodric left a comment

Choose a reason for hiding this comment

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

Please see comments

libs/mep/dc0944/aside/aside.js Outdated Show resolved Hide resolved
Co-authored-by: Vivian A Goodrich <101133187+vgoodric@users.noreply.github.com>
@aem-code-sync aem-code-sync bot temporarily deployed to dc0944stickybanner May 13, 2024 21:33 Inactive
@aem-code-sync aem-code-sync bot temporarily deployed to dc0944stickybanner May 13, 2024 23:39 Inactive
Copy link
Contributor

This pull request is not passing all required checks. Please see this discussion for information on how to get all checks passing. Inconsistent checks can be manually retried. If a test absolutely can not pass for a good reason, please add a comment with an explanation to the PR.

@aem-code-sync aem-code-sync bot temporarily deployed to dc0944stickybanner May 14, 2024 08:15 Inactive
libs/mep/dc0944/aside/aside.js Outdated Show resolved Hide resolved
libs/mep/dc0944/aside/aside.js Outdated Show resolved Hide resolved
libs/mep/dc0944/aside/aside.js Outdated Show resolved Hide resolved
libs/mep/dc0944/aside/aside.js Outdated Show resolved Hide resolved
libs/mep/dc0944/aside/aside.js Outdated Show resolved Hide resolved
libs/mep/dc0944/aside/aside.js Outdated Show resolved Hide resolved
libs/mep/dc0944/aside/aside.js Outdated Show resolved Hide resolved
Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>
@aem-code-sync aem-code-sync bot temporarily deployed to dc0944stickybanner May 14, 2024 16:56 Inactive
Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>
@aem-code-sync aem-code-sync bot temporarily deployed to dc0944stickybanner May 14, 2024 16:56 Inactive
Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>
@aem-code-sync aem-code-sync bot temporarily deployed to dc0944stickybanner May 14, 2024 16:57 Inactive
Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>
@aem-code-sync aem-code-sync bot temporarily deployed to dc0944stickybanner May 14, 2024 16:57 Inactive
Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>
@aem-code-sync aem-code-sync bot temporarily deployed to dc0944stickybanner May 14, 2024 17:00 Inactive
@aem-code-sync aem-code-sync bot temporarily deployed to dc0944stickybanner May 14, 2024 17:08 Inactive
@adobecom adobecom deleted a comment from robert-bogos May 14, 2024
@aem-code-sync aem-code-sync bot temporarily deployed to dc0944stickybanner May 14, 2024 17:26 Inactive
@aem-code-sync aem-code-sync bot temporarily deployed to dc0944stickybanner May 14, 2024 18:59 Inactive
@ivanvatadobe
Copy link
Contributor Author

@vgoodric I have made a single CSS update as a response to QA testing on milo preview pages.

The changes are committed and pushed. There are no other requests from QA.

@rgclayton
Copy link
Contributor

Why is this being submitted to the Milo repo? Shouldn't this POC code live under DC's repo?

cc: @vgoodric

@vgoodric
Copy link
Contributor

POC code live under DC

@rgclayton This on the Milo repo because it's using a version of aside for the test. It's quarantined to the MEP folder, so it doesn't really matter which repo it's on and it's easier to use the same repo that the block you're copying is also on. Less updates to the code.

@mokimo mokimo merged commit 8417da3 into stage May 16, 2024
11 checks passed
@mokimo mokimo deleted the dc0944stickybanner branch May 16, 2024 11:55
@mokimo mokimo mentioned this pull request May 16, 2024
mokimo added a commit that referenced this pull request May 16, 2024
* MWPW-147599: CaaS Bulk Publisher not working after MS authentication change (#2248)

Potential fix for bulk publisher issues

Co-authored-by: Blaine Gunn <Blainegunn@gmail.com>
Co-authored-by: Sheridan Sunier <sunier92@gmail.com>

* Preflight Accessibility tab update (#2254)

Updating how alt text is used and displayed in preflight

Co-authored-by: Ryan Clayton <rclayton@adobe.com>

* MWPW-146407: add new card style to caas configurator (#2279)

Co-authored-by: Brad Johnson <fullcolorcoder@gmail.com>
Co-authored-by: Sheridan Sunier <sunier@sheridans-mbp.corp.adobe.com>

* MWPW-141780 - Clean up Accordion block attributes (#2276)

remove role attribute on dd elements

Co-authored-by: Okan Sahin <39759830+mokimo@users.noreply.github.com>

* MWPW-147158: adds support for footer divider at the collection level (#2198)

* MWPW-147158: adds support for footer divider at the collection level

* Updatest unit-test files

* Update windows copy hash

* Update OSX copy hash

* Updates OSX hash again

* Removes console log

* Adds console logs for debbuging

* Removes console logs

* Adds console logs for debbuging again

* Removes console log

* Updates OSX hash

* Adds debuging message to test

* Update os-hash check logic

* trim() generated hash

* Changes the comparison logic for the hash

* Reverts last attempt

* new attempt to fix test

---------

Co-authored-by: Okan Sahin <39759830+mokimo@users.noreply.github.com>

* MWPW-147594: Consumers mappings for origin not being sent when using sidekick (#2244)

* Fix for origin always being upper cased

* Update tools/send-to-caas/send-utils.js

changing to const

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Okan Sahin <39759830+mokimo@users.noreply.github.com>

* Stage automation, update the body after merging another PR in (#2289)

* Stage process improvements

* Remove team reviewers

* Formatting

* Simplify PR bodies

---------

Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>

* Remove label requirements & merge workflows (#2288)

Remove label & merge workflows

Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>

* Quiz entry block (#2294)

* Initial quiz-entry block with ml field

* MWPW-144810: Quiz Entry - Add option cards and text to the block (#2095)

* Quiz entry block (#2103)

* MWPW-144810: Quiz Entry - Add option cards and text to the block

* MWPW-144810: Add the text to strings.xlsx for title, subtitle, ML field default, card instruction bar and the button

* MWPW-146243 - Quiz entry code optimization (#2121)

* MWPW-146243 - Quiz entry code optimization

* Restores code optimization that was lost in previous PRs

Resolves: [MWPW-146243](https://jira.corp.adobe.com/browse/MWPW-146243)

* Update utils.js

* Update quiz-entry.js

Set button to use string values

* Update quiz-entry.js

Fixed to use new strings object

* MWPW-146034 - Quiz entry block accessibility (#2139)

* resolved accessibility concerns when the ml input is used
* general code refinements for more clarity, specifically for getting string values

Resolves: [MWPW-146034](https://jira.corp.adobe.com/browse/MWPW-146034)

* MWPW-146036 - Rig up quiz entry button (#2190)

* Support for ml filtering
* Debug support using ?debug=quiz-entry
* Store quizState in local storage and redirect to the quiz

Resolves: [MWPW-146036](https://jira.corp.adobe.com/browse/MWPW-146036)

* Quiz entry block (#2204)

* MWPW-144810: Quiz Entry - Add option cards and text to the block

* MWPW-144810: Add the text to strings.xlsx for title, subtitle, ML field default, card instruction bar and the button

* MWPW-147031:Add Analytics for Quiz Entry Block, MWPW-146080: Integrate Auto-Complete for Open Text Field on UAR Exposed Front Door

* fix eslint

* fix code according feedbacks

* MWPW-144022: Quiz entry block (#2227)

* carousel starts

* got buttons working

* MWPW-144022: Prototype carousel refinement, keybord controls updated

* linting fixes

* MWPW-147482 - ML Input Bulletproofing (#2242)

* add support for fallback fi codes
* add lana logging for ml field failures
* fix so the redirect checks for maxQuestions

Resolves: [MWPW-147482](https://jira.corp.adobe.com/browse/MWPW-147482)

**Test URLs:**
- Before: https://main--milo--adobecom.hlx.page/?martech=off
- After: https://<branch>--milo--adobecom.hlx.page/?martech=off

* MWPW-147683 - CSS Cleanup (#2267)

* MWPW-147683 - CSS Cleanup

* Final pass on the css
* Edits to markup where necessary

Resolves: [MWPW-147683](https://jira.corp.adobe.com/browse/MWPW-147683)

* pr feedback for vars and eslint errors

* more PR feedback on icon placement

* pr feedback for button border

* cleaned up the carousel widths so it's consistent it all times as it was previously 8px short.

* pr feedback on the location of the input clear X.

* PR feedback - reduced border on input, fixed a card disable bug, addressed card layout in tablet

* carousel starts

* got buttons working

* MWPW-144022: Prototype carousel refinement, keybord controls updated

* linting fixes

* Tests

* working tests update

* Small font size fixes that design has asked for

* linting

* linting

* Update quiz-entry.js with default vals

* Bring back debug

* debugging debug

* Update test/blocks/quiz-entry/mocks/mock-data.js

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Update test/blocks/quiz-entry/mocks/mock-data.js

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Update test/blocks/quiz-entry/mocks/mock-data.js

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Update test/blocks/quiz-entry/mocks/mock-data.js

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Update test/blocks/quiz-entry/mocks/mock-data.js

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Update test/blocks/quiz-entry/mocks/mock-data.js

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Update test/blocks/quiz-entry/mocks/mock-data.js

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Update test/blocks/quiz-entry/mocks/mock-data.js

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Update test/blocks/quiz-entry/mocks/mock-data.js

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Update test/blocks/quiz-entry/quiz-entry.test.js

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Update test/blocks/quiz-entry/quiz-entry.test.js

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Update test/blocks/quiz-entry/mocks/mock-data.js

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Update test/blocks/quiz-entry/mocks/mock-data.js

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* added quiz-entry to utils.js (#2296)

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

* MWPW-148206: Update Spectra ML host from cchome-dev to cchome for PROD

---------

Co-authored-by: Jacky Sun <67350368+JackySun9@users.noreply.github.com>
Co-authored-by: Brad Johnson <fullcolorcoder@gmail.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Denys Fedotov <denlight@gmail.com>
Co-authored-by: Denys Fedotov <dfedotov@Denyss-MacBook-Pro.local>
Co-authored-by: Jacky Sun <XIASUN@ADOBE.COM>

* MWPW-147472: auto adjust commerce iframe (#2251)

* MWPW-147472: auto adjust commerce iframe.

* typo fix

* fix for potential bfcache issues

* improve modal selector query w.r.t hash

improve test.

---------

Co-authored-by: Mira Fedas <30750556+mirafedas@users.noreply.github.com>

* MWPW-141725 Graybox Block (#2282)

* Initial graybox overlay

* Add title & desc support

* Use more variables

* Add tests; fix issues found with tests

* Add blue design changes + body outline

* Fix blue border on device preview.  Small bug fixes.

* Only enable on -graybox urls or with graybox url param

* Update graybox check to use hostname instead of path

* Small bug fixes to handle when graybox url param already present

* Add check for localhost for unit tests

* PR feedback

* Add metadata check; code cov tweaks

* Change metadata setting

* add .page/.live selector

* Disable georouting modal when graybox is enabled

---------

Co-authored-by: Sunil Kamat <107644736+sukamat@users.noreply.github.com>
Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>

* MWPW-148253 Quiz Entry Coverage (#2305)

* Initial quiz-entry block with ml field

* MWPW-144810: Quiz Entry - Add option cards and text to the block (#2095)

* Quiz entry block (#2103)

* MWPW-144810: Quiz Entry - Add option cards and text to the block

* MWPW-144810: Add the text to strings.xlsx for title, subtitle, ML field default, card instruction bar and the button

* MWPW-146243 - Quiz entry code optimization (#2121)

* MWPW-146243 - Quiz entry code optimization

* Restores code optimization that was lost in previous PRs

Resolves: [MWPW-146243](https://jira.corp.adobe.com/browse/MWPW-146243)

* Update utils.js

* Update quiz-entry.js

Set button to use string values

* Update quiz-entry.js

Fixed to use new strings object

* MWPW-146034 - Quiz entry block accessibility (#2139)

* resolved accessibility concerns when the ml input is used
* general code refinements for more clarity, specifically for getting string values

Resolves: [MWPW-146034](https://jira.corp.adobe.com/browse/MWPW-146034)

* MWPW-146036 - Rig up quiz entry button (#2190)

* Support for ml filtering
* Debug support using ?debug=quiz-entry
* Store quizState in local storage and redirect to the quiz

Resolves: [MWPW-146036](https://jira.corp.adobe.com/browse/MWPW-146036)

* Quiz entry block (#2204)

* MWPW-144810: Quiz Entry - Add option cards and text to the block

* MWPW-144810: Add the text to strings.xlsx for title, subtitle, ML field default, card instruction bar and the button

* MWPW-147031:Add Analytics for Quiz Entry Block, MWPW-146080: Integrate Auto-Complete for Open Text Field on UAR Exposed Front Door

* fix eslint

* fix code according feedbacks

* MWPW-144022: Quiz entry block (#2227)

* carousel starts

* got buttons working

* MWPW-144022: Prototype carousel refinement, keybord controls updated

* linting fixes

* MWPW-147482 - ML Input Bulletproofing (#2242)

* add support for fallback fi codes
* add lana logging for ml field failures
* fix so the redirect checks for maxQuestions

Resolves: [MWPW-147482](https://jira.corp.adobe.com/browse/MWPW-147482)

**Test URLs:**
- Before: https://main--milo--adobecom.hlx.page/?martech=off
- After: https://<branch>--milo--adobecom.hlx.page/?martech=off

* MWPW-147683 - CSS Cleanup (#2267)

* MWPW-147683 - CSS Cleanup

* Final pass on the css
* Edits to markup where necessary

Resolves: [MWPW-147683](https://jira.corp.adobe.com/browse/MWPW-147683)

* pr feedback for vars and eslint errors

* more PR feedback on icon placement

* pr feedback for button border

* cleaned up the carousel widths so it's consistent it all times as it was previously 8px short.

* pr feedback on the location of the input clear X.

* PR feedback - reduced border on input, fixed a card disable bug, addressed card layout in tablet

* carousel starts

* got buttons working

* MWPW-144022: Prototype carousel refinement, keybord controls updated

* linting fixes

* Tests

* working tests update

* linting

* linting

* Update quiz-entry.js with default vals

* Bring back debug

* debugging debug

* Improved test coverage

* resolve conflicts

* restore style

* formatting fix

---------

Co-authored-by: Cody Lloyd <colloyd@adobe.com>
Co-authored-by: Jacky Sun <67350368+JackySun9@users.noreply.github.com>
Co-authored-by: Cody Lloyd <119891065+colloyd@users.noreply.github.com>

* Revert "MWPW-141725 Graybox Block (#2282)"

This reverts commit 7c47541.

* Revert "MWPW-148253 Quiz Entry Coverage (#2305)"

This reverts commit ed79630.

* Revert "Revert "MWPW-141725 Graybox Block (#2282)""

This reverts commit e14572a.

* MWPW-148253 Quiz Entry Coverage (#2305)

* Initial quiz-entry block with ml field

* MWPW-144810: Quiz Entry - Add option cards and text to the block (#2095)

* Quiz entry block (#2103)

* MWPW-144810: Quiz Entry - Add option cards and text to the block

* MWPW-144810: Add the text to strings.xlsx for title, subtitle, ML field default, card instruction bar and the button

* MWPW-146243 - Quiz entry code optimization (#2121)

* MWPW-146243 - Quiz entry code optimization

* Restores code optimization that was lost in previous PRs

Resolves: [MWPW-146243](https://jira.corp.adobe.com/browse/MWPW-146243)

* Update utils.js

* Update quiz-entry.js

Set button to use string values

* Update quiz-entry.js

Fixed to use new strings object

* MWPW-146034 - Quiz entry block accessibility (#2139)

* resolved accessibility concerns when the ml input is used
* general code refinements for more clarity, specifically for getting string values

Resolves: [MWPW-146034](https://jira.corp.adobe.com/browse/MWPW-146034)

* MWPW-146036 - Rig up quiz entry button (#2190)

* Support for ml filtering
* Debug support using ?debug=quiz-entry
* Store quizState in local storage and redirect to the quiz

Resolves: [MWPW-146036](https://jira.corp.adobe.com/browse/MWPW-146036)

* Quiz entry block (#2204)

* MWPW-144810: Quiz Entry - Add option cards and text to the block

* MWPW-144810: Add the text to strings.xlsx for title, subtitle, ML field default, card instruction bar and the button

* MWPW-147031:Add Analytics for Quiz Entry Block, MWPW-146080: Integrate Auto-Complete for Open Text Field on UAR Exposed Front Door

* fix eslint

* fix code according feedbacks

* MWPW-144022: Quiz entry block (#2227)

* carousel starts

* got buttons working

* MWPW-144022: Prototype carousel refinement, keybord controls updated

* linting fixes

* MWPW-147482 - ML Input Bulletproofing (#2242)

* add support for fallback fi codes
* add lana logging for ml field failures
* fix so the redirect checks for maxQuestions

Resolves: [MWPW-147482](https://jira.corp.adobe.com/browse/MWPW-147482)

**Test URLs:**
- Before: https://main--milo--adobecom.hlx.page/?martech=off
- After: https://<branch>--milo--adobecom.hlx.page/?martech=off

* MWPW-147683 - CSS Cleanup (#2267)

* MWPW-147683 - CSS Cleanup

* Final pass on the css
* Edits to markup where necessary

Resolves: [MWPW-147683](https://jira.corp.adobe.com/browse/MWPW-147683)

* pr feedback for vars and eslint errors

* more PR feedback on icon placement

* pr feedback for button border

* cleaned up the carousel widths so it's consistent it all times as it was previously 8px short.

* pr feedback on the location of the input clear X.

* PR feedback - reduced border on input, fixed a card disable bug, addressed card layout in tablet

* carousel starts

* got buttons working

* MWPW-144022: Prototype carousel refinement, keybord controls updated

* linting fixes

* Tests

* working tests update

* linting

* linting

* Update quiz-entry.js with default vals

* Bring back debug

* debugging debug

* Improved test coverage

* resolve conflicts

* restore style

* formatting fix

---------

Co-authored-by: Cody Lloyd <colloyd@adobe.com>
Co-authored-by: Jacky Sun <67350368+JackySun9@users.noreply.github.com>
Co-authored-by: Cody Lloyd <119891065+colloyd@users.noreply.github.com>

* Merge main into stage to resolve history issues (#2310)

* [Release] Stage to Main (#2299)

* MWPW-147599: CaaS Bulk Publisher not working after MS authentication change (#2248)

Potential fix for bulk publisher issues

Co-authored-by: Blaine Gunn <Blainegunn@gmail.com>
Co-authored-by: Sheridan Sunier <sunier92@gmail.com>

* Preflight Accessibility tab update (#2254)

Updating how alt text is used and displayed in preflight

Co-authored-by: Ryan Clayton <rclayton@adobe.com>

* MWPW-146407: add new card style to caas configurator (#2279)

Co-authored-by: Brad Johnson <fullcolorcoder@gmail.com>
Co-authored-by: Sheridan Sunier <sunier@sheridans-mbp.corp.adobe.com>

* MWPW-141780 - Clean up Accordion block attributes (#2276)

remove role attribute on dd elements

Co-authored-by: Okan Sahin <39759830+mokimo@users.noreply.github.com>

* MWPW-147158: adds support for footer divider at the collection level (#2198)

* MWPW-147158: adds support for footer divider at the collection level

* Updatest unit-test files

* Update windows copy hash

* Update OSX copy hash

* Updates OSX hash again

* Removes console log

* Adds console logs for debbuging

* Removes console logs

* Adds console logs for debbuging again

* Removes console log

* Updates OSX hash

* Adds debuging message to test

* Update os-hash check logic

* trim() generated hash

* Changes the comparison logic for the hash

* Reverts last attempt

* new attempt to fix test

---------

Co-authored-by: Okan Sahin <39759830+mokimo@users.noreply.github.com>

* MWPW-147594: Consumers mappings for origin not being sent when using sidekick (#2244)

* Fix for origin always being upper cased

* Update tools/send-to-caas/send-utils.js

changing to const

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Okan Sahin <39759830+mokimo@users.noreply.github.com>

* Stage automation, update the body after merging another PR in (#2289)

* Stage process improvements

* Remove team reviewers

* Formatting

* Simplify PR bodies

---------

Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>

* Remove label requirements & merge workflows (#2288)

Remove label & merge workflows

Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>

* Quiz entry block (#2294)

* Initial quiz-entry block with ml field

* MWPW-144810: Quiz Entry - Add option cards and text to the block (#2095)

* Quiz entry block (#2103)

* MWPW-144810: Quiz Entry - Add option cards and text to the block

* MWPW-144810: Add the text to strings.xlsx for title, subtitle, ML field default, card instruction bar and the button

* MWPW-146243 - Quiz entry code optimization (#2121)

* MWPW-146243 - Quiz entry code optimization

* Restores code optimization that was lost in previous PRs

Resolves: [MWPW-146243](https://jira.corp.adobe.com/browse/MWPW-146243)

* Update utils.js

* Update quiz-entry.js

Set button to use string values

* Update quiz-entry.js

Fixed to use new strings object

* MWPW-146034 - Quiz entry block accessibility (#2139)

* resolved accessibility concerns when the ml input is used
* general code refinements for more clarity, specifically for getting string values

Resolves: [MWPW-146034](https://jira.corp.adobe.com/browse/MWPW-146034)

* MWPW-146036 - Rig up quiz entry button (#2190)

* Support for ml filtering
* Debug support using ?debug=quiz-entry
* Store quizState in local storage and redirect to the quiz

Resolves: [MWPW-146036](https://jira.corp.adobe.com/browse/MWPW-146036)

* Quiz entry block (#2204)

* MWPW-144810: Quiz Entry - Add option cards and text to the block

* MWPW-144810: Add the text to strings.xlsx for title, subtitle, ML field default, card instruction bar and the button

* MWPW-147031:Add Analytics for Quiz Entry Block, MWPW-146080: Integrate Auto-Complete for Open Text Field on UAR Exposed Front Door

* fix eslint

* fix code according feedbacks

* MWPW-144022: Quiz entry block (#2227)

* carousel starts

* got buttons working

* MWPW-144022: Prototype carousel refinement, keybord controls updated

* linting fixes

* MWPW-147482 - ML Input Bulletproofing (#2242)

* add support for fallback fi codes
* add lana logging for ml field failures
* fix so the redirect checks for maxQuestions

Resolves: [MWPW-147482](https://jira.corp.adobe.com/browse/MWPW-147482)

**Test URLs:**
- Before: https://main--milo--adobecom.hlx.page/?martech=off
- After: https://<branch>--milo--adobecom.hlx.page/?martech=off

* MWPW-147683 - CSS Cleanup (#2267)

* MWPW-147683 - CSS Cleanup

* Final pass on the css
* Edits to markup where necessary

Resolves: [MWPW-147683](https://jira.corp.adobe.com/browse/MWPW-147683)

* pr feedback for vars and eslint errors

* more PR feedback on icon placement

* pr feedback for button border

* cleaned up the carousel widths so it's consistent it all times as it was previously 8px short.

* pr feedback on the location of the input clear X.

* PR feedback - reduced border on input, fixed a card disable bug, addressed card layout in tablet

* carousel starts

* got buttons working

* MWPW-144022: Prototype carousel refinement, keybord controls updated

* linting fixes

* Tests

* working tests update

* Small font size fixes that design has asked for

* linting

* linting

* Update quiz-entry.js with default vals

* Bring back debug

* debugging debug

* Update test/blocks/quiz-entry/mocks/mock-data.js

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Update test/blocks/quiz-entry/mocks/mock-data.js

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Update test/blocks/quiz-entry/mocks/mock-data.js

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Update test/blocks/quiz-entry/mocks/mock-data.js

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Update test/blocks/quiz-entry/mocks/mock-data.js

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Update test/blocks/quiz-entry/mocks/mock-data.js

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Update test/blocks/quiz-entry/mocks/mock-data.js

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Update test/blocks/quiz-entry/mocks/mock-data.js

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Update test/blocks/quiz-entry/mocks/mock-data.js

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Update test/blocks/quiz-entry/quiz-entry.test.js

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Update test/blocks/quiz-entry/quiz-entry.test.js

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Update test/blocks/quiz-entry/mocks/mock-data.js

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Update test/blocks/quiz-entry/mocks/mock-data.js

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* added quiz-entry to utils.js (#2296)

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

* MWPW-148206: Update Spectra ML host from cchome-dev to cchome for PROD

---------

Co-authored-by: Jacky Sun <67350368+JackySun9@users.noreply.github.com>
Co-authored-by: Brad Johnson <fullcolorcoder@gmail.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Denys Fedotov <denlight@gmail.com>
Co-authored-by: Denys Fedotov <dfedotov@Denyss-MacBook-Pro.local>
Co-authored-by: Jacky Sun <XIASUN@ADOBE.COM>

---------

Co-authored-by: Sanjay Rai <sanrai@adobe.com>
Co-authored-by: Blaine Gunn <Blainegunn@gmail.com>
Co-authored-by: Sheridan Sunier <sunier92@gmail.com>
Co-authored-by: Ryan Clayton <rgclayton@gmail.com>
Co-authored-by: Ryan Clayton <rclayton@adobe.com>
Co-authored-by: Brad Johnson <fullcolorcoder@gmail.com>
Co-authored-by: Sheridan Sunier <sunier@sheridans-mbp.corp.adobe.com>
Co-authored-by: Elan Bartholomew <79870969+elan-tbx@users.noreply.github.com>
Co-authored-by: Okan Sahin <39759830+mokimo@users.noreply.github.com>
Co-authored-by: cmiqueo <64917520+cmiqueo@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>
Co-authored-by: Cody Lloyd <119891065+colloyd@users.noreply.github.com>
Co-authored-by: Jacky Sun <67350368+JackySun9@users.noreply.github.com>
Co-authored-by: Denys Fedotov <denlight@gmail.com>
Co-authored-by: Denys Fedotov <dfedotov@Denyss-MacBook-Pro.local>
Co-authored-by: Jacky Sun <XIASUN@ADOBE.COM>

* Revert "MWPW-141725 Graybox Block (#2282)"

This reverts commit 7c47541.

* Revert "MWPW-148253 Quiz Entry Coverage (#2305)"

This reverts commit ed79630.

* Revert "Revert "MWPW-141725 Graybox Block (#2282)""

This reverts commit e14572a.

* MWPW-148253 Quiz Entry Coverage (#2305)

* Initial quiz-entry block with ml field

* MWPW-144810: Quiz Entry - Add option cards and text to the block (#2095)

* Quiz entry block (#2103)

* MWPW-144810: Quiz Entry - Add option cards and text to the block

* MWPW-144810: Add the text to strings.xlsx for title, subtitle, ML field default, card instruction bar and the button

* MWPW-146243 - Quiz entry code optimization (#2121)

* MWPW-146243 - Quiz entry code optimization

* Restores code optimization that was lost in previous PRs

Resolves: [MWPW-146243](https://jira.corp.adobe.com/browse/MWPW-146243)

* Update utils.js

* Update quiz-entry.js

Set button to use string values

* Update quiz-entry.js

Fixed to use new strings object

* MWPW-146034 - Quiz entry block accessibility (#2139)

* resolved accessibility concerns when the ml input is used
* general code refinements for more clarity, specifically for getting string values

Resolves: [MWPW-146034](https://jira.corp.adobe.com/browse/MWPW-146034)

* MWPW-146036 - Rig up quiz entry button (#2190)

* Support for ml filtering
* Debug support using ?debug=quiz-entry
* Store quizState in local storage and redirect to the quiz

Resolves: [MWPW-146036](https://jira.corp.adobe.com/browse/MWPW-146036)

* Quiz entry block (#2204)

* MWPW-144810: Quiz Entry - Add option cards and text to the block

* MWPW-144810: Add the text to strings.xlsx for title, subtitle, ML field default, card instruction bar and the button

* MWPW-147031:Add Analytics for Quiz Entry Block, MWPW-146080: Integrate Auto-Complete for Open Text Field on UAR Exposed Front Door

* fix eslint

* fix code according feedbacks

* MWPW-144022: Quiz entry block (#2227)

* carousel starts

* got buttons working

* MWPW-144022: Prototype carousel refinement, keybord controls updated

* linting fixes

* MWPW-147482 - ML Input Bulletproofing (#2242)

* add support for fallback fi codes
* add lana logging for ml field failures
* fix so the redirect checks for maxQuestions

Resolves: [MWPW-147482](https://jira.corp.adobe.com/browse/MWPW-147482)

**Test URLs:**
- Before: https://main--milo--adobecom.hlx.page/?martech=off
- After: https://<branch>--milo--adobecom.hlx.page/?martech=off

* MWPW-147683 - CSS Cleanup (#2267)

* MWPW-147683 - CSS Cleanup

* Final pass on the css
* Edits to markup where necessary

Resolves: [MWPW-147683](https://jira.corp.adobe.com/browse/MWPW-147683)

* pr feedback for vars and eslint errors

* more PR feedback on icon placement

* pr feedback for button border

* cleaned up the carousel widths so it's consistent it all times as it was previously 8px short.

* pr feedback on the location of the input clear X.

* PR feedback - reduced border on input, fixed a card disable bug, addressed card layout in tablet

* carousel starts

* got buttons working

* MWPW-144022: Prototype carousel refinement, keybord controls updated

* linting fixes

* Tests

* working tests update

* linting

* linting

* Update quiz-entry.js with default vals

* Bring back debug

* debugging debug

* Improved test coverage

* resolve conflicts

* restore style

* formatting fix

---------

Co-authored-by: Cody Lloyd <colloyd@adobe.com>
Co-authored-by: Jacky Sun <67350368+JackySun9@users.noreply.github.com>
Co-authored-by: Cody Lloyd <119891065+colloyd@users.noreply.github.com>

---------

Co-authored-by: Brad Johnson <fullcolorcoder@gmail.com>
Co-authored-by: milo-pr-merge[bot] <169241390+milo-pr-merge[bot]@users.noreply.github.com>
Co-authored-by: Sanjay Rai <sanrai@adobe.com>
Co-authored-by: Blaine Gunn <Blainegunn@gmail.com>
Co-authored-by: Sheridan Sunier <sunier92@gmail.com>
Co-authored-by: Ryan Clayton <rgclayton@gmail.com>
Co-authored-by: Ryan Clayton <rclayton@adobe.com>
Co-authored-by: Sheridan Sunier <sunier@sheridans-mbp.corp.adobe.com>
Co-authored-by: Elan Bartholomew <79870969+elan-tbx@users.noreply.github.com>
Co-authored-by: cmiqueo <64917520+cmiqueo@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>
Co-authored-by: Cody Lloyd <119891065+colloyd@users.noreply.github.com>
Co-authored-by: Jacky Sun <67350368+JackySun9@users.noreply.github.com>
Co-authored-by: Denys Fedotov <denlight@gmail.com>
Co-authored-by: Denys Fedotov <dfedotov@Denyss-MacBook-Pro.local>
Co-authored-by: Jacky Sun <XIASUN@ADOBE.COM>
Co-authored-by: Cody Lloyd <colloyd@adobe.com>

* Revert "MWPW-147472: auto adjust commerce iframe" (#2312)

Revert "MWPW-147472: auto adjust commerce iframe (#2251)"

This reverts commit 481bbcb.

* OPT-26580 DC0994_US | Frictionless | Add sticky banner Upload CTA (#2298)

* 05-13-24 ivan dc0944 first

* Update libs/mep/dc0944/aside/aside.js

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

* 05-13-24 ivan dc0944 add css changes and js optimization

* Update libs/mep/dc0944/aside/aside.js

Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>

* Update libs/mep/dc0944/aside/aside.js

Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>

* Update libs/mep/dc0944/aside/aside.js

Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>

* Update libs/mep/dc0944/aside/aside.js

Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>

* Update libs/mep/dc0944/aside/aside.js

Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>

* 05-14-24 ivan renamed mep test folder dc0994

* 05-14-24 ivan fix sticky banner close btn

---------

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

---------

Co-authored-by: Sanjay Rai <sanrai@adobe.com>
Co-authored-by: Blaine Gunn <Blainegunn@gmail.com>
Co-authored-by: Sheridan Sunier <sunier92@gmail.com>
Co-authored-by: Ryan Clayton <rgclayton@gmail.com>
Co-authored-by: Ryan Clayton <rclayton@adobe.com>
Co-authored-by: Brad Johnson <fullcolorcoder@gmail.com>
Co-authored-by: Sheridan Sunier <sunier@sheridans-mbp.corp.adobe.com>
Co-authored-by: Elan Bartholomew <79870969+elan-tbx@users.noreply.github.com>
Co-authored-by: Okan Sahin <39759830+mokimo@users.noreply.github.com>
Co-authored-by: cmiqueo <64917520+cmiqueo@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>
Co-authored-by: Cody Lloyd <119891065+colloyd@users.noreply.github.com>
Co-authored-by: Jacky Sun <67350368+JackySun9@users.noreply.github.com>
Co-authored-by: Denys Fedotov <denlight@gmail.com>
Co-authored-by: Denys Fedotov <dfedotov@Denyss-MacBook-Pro.local>
Co-authored-by: Jacky Sun <XIASUN@ADOBE.COM>
Co-authored-by: ilyas Stéphane Türkben <tuerkben@adobe.com>
Co-authored-by: Mira Fedas <30750556+mirafedas@users.noreply.github.com>
Co-authored-by: Chris Peyer <chrischrischris@users.noreply.github.com>
Co-authored-by: Sunil Kamat <107644736+sukamat@users.noreply.github.com>
Co-authored-by: Cody Lloyd <colloyd@adobe.com>
Co-authored-by: Okan Sahin <osahin@adobe.com>
Co-authored-by: milo-pr-merge[bot] <169241390+milo-pr-merge[bot]@users.noreply.github.com>
Co-authored-by: Jan Ivan Viloria <158211769+ivanvatadobe@users.noreply.github.com>
Co-authored-by: Vivian A Goodrich <101133187+vgoodric@users.noreply.github.com>
Co-authored-by: markpadbe <markp@adobe.com>
@milo-pr-merge milo-pr-merge bot mentioned this pull request May 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
high priority Why is this a high priority? Blocker? Critical? Dependency? Ready for Stage run-nala Run Nala Test Automation against PR verified PR has been E2E tested by a reviewer zero-impact
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants