Skip to content

fix: redirect admitted users away from beta wait page#137

Draft
Copilot wants to merge 2 commits intomainfrom
copilot/fix-waiting-for-approval-page
Draft

fix: redirect admitted users away from beta wait page#137
Copilot wants to merge 2 commits intomainfrom
copilot/fix-waiting-for-approval-page

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 25, 2026

Users granted beta access while sitting on /beta/wait/ would remain stuck there on reload — the view unconditionally rendered the waiting page regardless of current admission status.

Changes

  • views.pybeta_wait: Call is_beta_admitted() before rendering; redirect to overview if the user is already admitted.
def beta_wait(request: HttpRequest) -> HttpResponse:
    if is_beta_admitted(request.user):
        return redirect(reverse("overview"))
    return render(request, "toxtempass/beta_wait.html")
  • tests/test_admin_beta.py: Two new cases — pending user still sees the wait page; admitted user is redirected to overview.

Copilot AI linked an issue Apr 25, 2026 that may be closed by this pull request
Copilot AI changed the title [WIP] Fix redirection after approval from waiting page fix: redirect admitted users away from beta wait page Apr 25, 2026
Copilot AI requested a review from johannehouweling April 25, 2026 17:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Waiting for approval page

2 participants