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

Release v0.8.0.beta.1 (Sprint 42) #1890

Merged
merged 186 commits into from
Oct 21, 2024
Merged

Release v0.8.0.beta.1 (Sprint 42) #1890

merged 186 commits into from
Oct 21, 2024

Conversation

scottqueen-bixal
Copy link
Collaborator

@scottqueen-bixal scottqueen-bixal commented Oct 21, 2024

PR Summary

Related Github Issues

Application Improvements

NEW FEATURE | Return to first step of form, without clearing the form #1473
NEW FEATURE | client side routing in embedded applications #1831
TEST | NGINX redirects for client side routing fallback #1838
Form-page-2 page change event firing twice #1786
Remove crazyegg script value updates from application #1811

Custom Module Improvements

Enable worflow for Benefit Finder nodes that can be in draft #1710
No JSON file generation for moderation state Needs Review #1846
prevent agency archived if still used #1847
Do not use archived content in building JSON data #1858
display raw html #1817

Detailed Testing steps

NEW FEATURE | Return to first step of form, without clearing the form #1473
NEW FEATURE | client side routing in embedded applications #1831

Detailed Testing steps

  • update nginx config,
  • add the following redirects to usagov-2021/.docker/src-cms/etc/nginx/partials/internal_redirects.conf
 rewrite ^/benefit-finder/([^/]+)/[^/]+$ /benefit-finder/$1 permanent;
 rewrite ^/es/buscador-beneficios/([^/]+)/[^/]+$ /es/buscador-beneficios/$1 permanent;
  • uncomment include partials/internal_redirects.conf; in usagov-2021/.docker/src-cms/etc/nginx/partials/cms.conf.tmpl
    include partials/drupal.conf;
    ## un-comment this to test internal redirects
    ## (which should match the corresponding internal_redirects.conf in src-www):
    include partials/internal_redirects.conf;
  • rebuild docker containers
  • navigate to /benefit-finder/death on localhost

NEW FEATURE | client side routing in embedded applications #1831

  • CLICK 'Start...`
  • include values for required fields for both form steps
  • CLICK the back button in the browser
  • ensure user can navigate back to beginning of form
  • CLICK the forward button in the browser
  • ensure user can navigate back to second step of form
  • ensure data entered is still present in values of fieldsets
  • CLICK continue
  • CLICK "Get your results"
  • ensure zero benefits view
  • re-test in es

NEW FEATURE | Return to first step of form, without clearing the form #1473

  • CLICK "Go back to form"
  • ensure you are on step one of form
  • answer yes to radios
  • in modal, CLICK view "Review your selections"
  • CLICK "Next"
  • Ensure eligible results view
  • CLICK "Explore other potential benefits"
  • Ensure not-eligible view
  • CLICK the back button in the browser
  • Ensure eligible results view
  • CLICK `Share link"
  • paste link in browser URL
  • ensure alert for unsaved changes
Screenshot 2024-10-10 at 11 22 52 AM
  • Ensure eligible results view
  • CLICK "Clear my answers..."

TEST | NGINX redirects for client side routing fallback #1838

  • Ensure intro view
  • CLICK 'Start...`
  • Ensure child path of life event is form step heading, ie. /death/about-you
  • enter data for first form question
  • refresh page
  • ensure alert for unsaved changes
  • Ensure intro view

Form-page-2 page change event firing twice #1786

  • navigate to /disability
  • CLICK start
  • CLICK next
  • resolve error
  • CLICK next
  • CLICK next
  • CLICK verify
  • console.log(window.dataLayer)
  • ensure that bf_page_change event for modal and form page 2 only fired once
Screenshot 2024-10-10 at 11 45 01 AM

Custom Module Improvements

Enable worflow for Benefit Finder nodes that can be in draft #1710

  • Navigate to node/add
  • CLICK Agency
  • Create an agency
  • Save as "Draft
  • CLICK "Edit"
  • Save as "Needs Review"
  • CLICK "Edit"
  • Save as Published
  • CLICK "Edit"
  • Save as Archived
  • CLICK "Edit"
  • Save as Draft

No JSON file generation for moderation state Needs Review #1846

  • Navigate to admin/content?combine=&type=bears_agency&status=All&langcode=All
  • Go to agency "Department of Education (DOE)" edit page
  • Change to Draft
  • CLICK Save
  • Verify JSON file generation
  • Go to agency "Department of Education (DOE)" edit page
  • Change to Needs Review
  • CLICK Save
  • Verify no JSON file generation
  • Verify seeing warning message

prevent agency archived if still used #1847

  • Navigate to admin/content?combine=&type=bears_agency&status=All&langcode=All
  • Go to agency "Social Security Administration (SSA)" edit page
  • Change to Archived (unpublished)
  • CLICK Save button
  • Verify error message with list of benefits that use the agency

Do not use archived content in building JSON data #1858

To test in local development site or in dev site.

  • Navigate to benefit-finder/api/life-event/death
  • Verify benefit "COVID-19 funeral assistance" in JSON data
  • Navigate to admin/content?combine=COVID&type=bears_benefit&status=All&langcode=All
  • Go to benefit "COVID-19 funeral assistance" edit page
  • Change to archived
  • CLICK Save
  • Navigate to benefit-finder/api/life-event/death
  • Verify no benefit "COVID-19 funeral assistance" in JSON data

display raw html #1817

To test in local development site or in dev site.

  • navigate to admin/content?combine=&type=bears_life_event_form&status=All&langcode=All
  • go to life event form "Benefit finder: retirement" edit page
  • add <a href="http://usagov.gov">USAGov</a> in summary field
  • save as published
  • navigate to benefit-finder/api/life-event/retirement
  • verify the hyperlink in summary as raw html

nehemiah-abuga and others added 30 commits September 16, 2024 07:00
PXBF-1703-clean-cy-script-commands: clean up cy script commands
@scottqueen-bixal scottqueen-bixal marked this pull request as ready for review October 21, 2024 17:35
Copy link
Collaborator

@XavierMetichecchia XavierMetichecchia left a comment

Choose a reason for hiding this comment

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

Approved

@scottqueen-bixal scottqueen-bixal merged commit 0a08218 into main Oct 21, 2024
15 checks passed
@scottqueen-bixal scottqueen-bixal changed the title Release v0.8.0.beta.1 (Sprint 42) Release v0.8.0.beta.1 (Sprint 41) Oct 30, 2024
@scottqueen-bixal scottqueen-bixal changed the title Release v0.8.0.beta.1 (Sprint 41) Release v0.8.0.beta.1 (Sprint 42) Oct 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants