LG-6577: Show error page when FSMv2 React application crashes#6509
LG-6577: Show error page when FSMv2 React application crashes#6509
Conversation
app/javascript/packs/verify-flow.tsx
Outdated
There was a problem hiding this comment.
It would be nice to handle this logic inside the components so that we can have ErrorBoundary dealing with all errors and not have to duplicate as much. Not sure how straight-forward a refactor that would be and if it'd be best to do here vs. a separate pull request.
**Why**: As a user, I expect that I am informed if there is an issue with the application, so that I know what my options are for continuing. changelog: Upcoming Features, Identity Verification, Add application crash handling error page
aduth
force-pushed
the
aduth-lg-6577-fsmv2-react-crash
branch
from
aa3147d to
4de30e2
Compare
| }, | ||
| ]} | ||
| /> | ||
| {inPersonURL && ( |
There was a problem hiding this comment.
cc @tomas-nava @sheldon-b looks like we're adding one more page that links to IPP, something to consider when adding the intro + location select in React
There was a problem hiding this comment.
I thought it might be nice to consolidate this with DocumentCaptureTroubleshootingOptions, but as it stands, the options are pretty different than the ones we have there.
I suppose another idea could be to at least extract a common IPP troubleshooting options to share between these two and any future usage, if that would be helpful.
2 participants
Why: As a user, I expect that I am informed if there is an issue with the application, so that I know what my options are for continuing.
Screenshot:
Testing Instructions:
Before: You see a blank page upon reaching the "Please re-enter your password" step
After: You see the error page as in the screenshot above