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

Remove load & continue feature #55

Merged
merged 1 commit into from
Oct 8, 2024
Merged

Remove load & continue feature #55

merged 1 commit into from
Oct 8, 2024

Conversation

maxatdetroit
Copy link
Member

@maxatdetroit maxatdetroit commented Oct 7, 2024

Context

We want to remove the load & save feature for the property application because the API will be changing this week.

Changes in this PR

  • Remove the load & save feature for the property application
  • Update the user disclaimers to remove notices about the load & save feature and how the application ID can be used
  • Remove the load & save feature GET HTTP request logic for the digital divide survey (which is no longer active anyway)
    • Note: In a future PR (Remove survey logic and data #56), I'll remove all the digital divide survey data and logic because the survey POST and GET APIs will also be changing and would break this survey if it were to relaunch.

Note: While the UX treatment has been updated slightly to remove the info about the load & continue feature, the UX flow is still a little broken in the event someone does not complete their application and tries to check the status. This should be fixed in a future PR. I wrote an issue here: #54.

Testing

Screen.Recording.2024-10-07.at.11.06.01.AM.mov

@maxatdetroit maxatdetroit added the bug Something isn't working label Oct 7, 2024
@maxatdetroit maxatdetroit self-assigned this Oct 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants