Skip to content

fix(setup): make sure CAN_INSTALL error page is shown - #62765

Open
joshtrichards wants to merge 1 commit into
masterfrom
jtr/fix-setup-canInstall-failure-page
Open

fix(setup): make sure CAN_INSTALL error page is shown#62765
joshtrichards wants to merge 1 commit into
masterfrom
jtr/fix-setup-canInstall-failure-page

Conversation

@joshtrichards

@joshtrichards joshtrichards commented Aug 1, 2026

Copy link
Copy Markdown
Member
  • Resolves: #

Summary

printGuestPage() doesn't terminate after displaying a template. The error page is effectively invisible due to the redirect immediately after.

Stop finishSetup() from redirecting after displaying the CAN_INSTALL cleanup error page, so users see the actionable message.

TODO

  • ...

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

Signed-off-by: Josh <josh.t.richards@gmail.com>
@joshtrichards joshtrichards added this to the Nextcloud 35 milestone Aug 1, 2026
@joshtrichards
joshtrichards requested a review from a team as a code owner August 1, 2026 15:59
@joshtrichards joshtrichards added bug 3. to review Waiting for reviews labels Aug 1, 2026
@joshtrichards
joshtrichards removed the request for review from a team August 1, 2026 15:59
@joshtrichards joshtrichards changed the title fix(setup): error handling when CAN_INSTALL cannot be removed fix(setup): make sure CAN_INSTALL error page is shown Aug 1, 2026
@joshtrichards

Copy link
Copy Markdown
Member Author

/backport to stable34

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant