If a PostRequireEvent subscriber throws an exception, users are redirected to the confirmation page

Created on 17 April 2023, over 1 year ago

Problem/Motivation

This is yet another bug discovered while working on πŸ“Œ Add functional test that proves there is reasonable UX whenever a stage event subscriber has an exception Fixed .

It turns out that, if a PostRequireEvent subscriber throws an exception, the user is still redirect to the confirmation page to continue the update (/admin/automatic-update-ready). They will see the error message, and they won't be able to continue the update, but what's bad here is that this is very incongruous UX, because if a PreRequireEvent subscriber throws an exception, users are redirected to the start page!

Proposed resolution

If a PostRequireEvent subscriber throws an exception, the stage should be destroyed and the user should be redirected back to the start.

πŸ› Bug report
Status

Closed: duplicate

Version

3.0

Component

Code

Created by

πŸ‡ΊπŸ‡ΈUnited States phenaproxima Massachusetts

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Production build 0.71.5 2024