Skip to content

fix(teams): add /accept path to invitation email URL#252

Open
tturnerdev wants to merge 1 commit into
litestar-org:mainfrom
tturnerdev:fix/invitation-url
Open

fix(teams): add /accept path to invitation email URL#252
tturnerdev wants to merge 1 commit into
litestar-org:mainfrom
tturnerdev:fix/invitation-url

Conversation

@tturnerdev

Copy link
Copy Markdown
Contributor

Summary

  • The team invitation email links to /teams/{id}/invitations/{id} but the frontend accept page route is at /teams/{id}/invitations/{id}/accept
  • Without the /accept suffix, clicking the link loads the team detail page (or redirects to /home if the user isn't a member), instead of showing the accept/decline UI

Test plan

  • Invite a user to a team
  • Open the invitation email in Mailpit
  • Verify the URL ends with /accept
  • Click the link — should show the accept/decline invitation page

🤖 Generated with Claude Code

The invitation email linked to /teams/{id}/invitations/{id} but the
frontend route for accepting is at .../accept. Without the suffix,
the URL falls through to the team detail page.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@tturnerdev tturnerdev requested review from a team and cofin as code owners April 28, 2026 05:49
@sonarqubecloud

Copy link
Copy Markdown

@tturnerdev

Copy link
Copy Markdown
Contributor Author

PR created by AI; reviewed and verified by me.

Here's a screenshot of the invite email in Mailpit:
image

Example invite URL from screenshot:

http://localhost:8000/teams/019dd205-5ce4-7681-8235-198edf549ec9/invitations/019dd23a-8d1e-7351-a382-16ab3206abcd

Navigating to this URL resulted in either:

  1. Already logged in: redirect to /home (http://localhost/home); or
  2. Not logged in: redirect to to /login (http://localhost/home)

Additionally, in my testing, when appending /accept to the URL, it does appear the URL redirected correctly, however, I ran into 403 errors (assuming due to the /teams/[id] route being permissioned).

This issue is addressed separately in #254

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.

2 participants