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

Clicking "Start over" results in duplicate Resume modal #186

Open
2 tasks
Hoolean opened this issue Oct 2, 2019 · 1 comment
Open
2 tasks

Clicking "Start over" results in duplicate Resume modal #186

Hoolean opened this issue Oct 2, 2019 · 1 comment
Labels
bug wontfix Not in current scope

Comments

@Hoolean
Copy link

Hoolean commented Oct 2, 2019

Problem

When a user is greeted with the resume modal (at /#resume, labelled "Hello again"), pressing "Start over" just leads them back to the same dialog. They must press it a further time to reach the home page.

Further Implications

The dialog occasionally appears unexpectedly when running Grunt tests for SEAP ESA, failing them. I suspect this could be caused by the same issue.

Reproducing

  1. Navigate to the live SEAP ESA website
  2. If you are not presented with the modal:
    1. Answer some questions on the site
    2. Close your tab
    3. Navigate to the site again

Expected

The user is taken to the home page.

image

Actual

The user is returned to the modal.

image

Note that the URL has still changed from http://www.esa-assessment.support/#resume to http://www.esa-assessment.support/#main-menu.

Further Thoughts

  • Is the other button functioning correctly?
  • Should the button do anything other than redirect the user to the main menu?

Environment

  • Firefox v69.0.1 (admittedly with some add-ons)
  • Ubuntu 19.04
@Hoolean Hoolean added the bug label Oct 2, 2019
@katjam
Copy link
Contributor

katjam commented Oct 4, 2019

I think we have to leave any flow stuff for a rewrite at this point. Historically fixing one part of flow broke other parts.

@katjam katjam added the wontfix Not in current scope label Oct 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug wontfix Not in current scope
Projects
None yet
Development

No branches or pull requests

2 participants