Skip to content

PM-4645: route invited project access to invitation modal#1576

Merged
jmgasper merged 2 commits intodevfrom
PM-4645
Apr 1, 2026
Merged

PM-4645: route invited project access to invitation modal#1576
jmgasper merged 2 commits intodevfrom
PM-4645

Conversation

@jmgasper
Copy link
Copy Markdown
Collaborator

@jmgasper jmgasper commented Apr 1, 2026

What was broken
Opening an invited project from the Work Manager projects list always routed users to the project challenges page, so the join/decline invitation modal did not appear.

Root cause
Project list navigation always used the default challenges path and never checked whether the current user was only a pending invitee for that project.

What was changed
Added an invite-aware project landing path helper that reuses the existing invite ownership matcher, and updated the projects table/card links to send invited users to /projects/:projectId/invitations instead of /projects/:projectId/challenges.

Any added/updated tests
Added navigation.utils.spec.ts coverage for invited-user and non-invited-user project landing paths, and kept the related permissions utils tests passing.


Open with Devin

What was broken
Opening an invited project from the Work Manager projects list always routed users to the project challenges page, so the join/decline invitation modal did not appear.

Root cause
Project list navigation always used the default challenges path and never checked whether the current user was only a pending invitee for that project.

What was changed
Added an invite-aware project landing path helper that reuses the existing invite ownership matcher, and updated the projects table/card links to send invited users to /projects/:projectId/invitations instead of /projects/:projectId/challenges.

Any added/updated tests
Added navigation.utils.spec.ts coverage for invited-user and non-invited-user project landing paths, and kept the related permissions utils tests passing.
@jmgasper jmgasper requested a review from kkartunov as a code owner April 1, 2026 13:57
devin-ai-integration[bot]

This comment was marked as resolved.

@jmgasper jmgasper merged commit ca51394 into dev Apr 1, 2026
9 checks passed
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.

1 participant