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

quiz entry block - fix for content disappearing before redirect #2348

Merged
merged 1 commit into from
May 21, 2024

Conversation

colloyd
Copy link
Contributor

@colloyd colloyd commented May 21, 2024

  • bug fix for content flicker before redirect
  • bug fix for debug mode where you cannot continue more than once

Resolves: MWPW-149356

Test URLs:

Testing notes: To test the flicker and the fix, turn bandwidth throttling down in dev tools. Make a selection and press continue. Before, the content would disappear just before the page redirects to the quiz page. After, the content persists until the redirect happens.

To test the debug mode fix, add the debug=quiz-entry query param and make a selection and press continue. Alter your selection and press continue again. Before, it would throw a script error. After the script error is no longer present and your new selection logging works as expected.

* bug fix for content flicker before redirect
* bug fix for debug mode where you cannot continue more than once

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

**Test URLs:**
- Before: https://main--milo--adobecom.hlx.page/?martech=off
- After: https://<branch>--milo--adobecom.hlx.page/?martech=off
@colloyd colloyd added the needs-verification PR requires E2E testing by a reviewer label May 21, 2024
Copy link
Contributor

aem-code-sync bot commented May 21, 2024

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

@JackySun9 JackySun9 added verified PR has been E2E tested by a reviewer new-feature New block or other feature high priority Why is this a high priority? Blocker? Critical? Dependency? uar Ready for Stage and removed needs-verification PR requires E2E testing by a reviewer labels May 21, 2024
@Blainegunn Blainegunn merged commit 8899363 into adobecom:stage May 21, 2024
15 checks passed
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? new-feature New block or other feature Ready for Stage uar verified PR has been E2E tested by a reviewer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants