Skip to content

feat: add ?tab=project alias for ?tab=team#521

Merged
mishushakov merged 1 commit into
private/mainfrom
devin/1784840497-tab-project-alias
Jul 24, 2026
Merged

feat: add ?tab=project alias for ?tab=team#521
mishushakov merged 1 commit into
private/mainfrom
devin/1784840497-tab-project-alias

Conversation

@devin-ai-integration

Copy link
Copy Markdown
Contributor

Summary

Adds a project key to TAB_URL_MAP so /dashboard?tab=project resolves to the same destination as /dashboard?tab=team (the General settings page). This follows the teams→projects rename in user-visible copy, giving external crosslinks (docs, CLI, emails) a project entrypoint that matches the new terminology.

  team:    (teamSlug) => PROTECTED_URLS.GENERAL(teamSlug),
+ project: (teamSlug) => PROTECTED_URLS.GENERAL(teamSlug),
  general: (teamSlug) => PROTECTED_URLS.GENERAL(teamSlug),

The existing team mapping is kept for backwards compatibility. The integration test in dashboard-route.test.ts iterates TAB_URL_MAP dynamically, so the new tab is covered automatically (22/22 passing).

Link to Devin session: https://app.devin.ai/sessions/3445ffca3086454aa4dcc585198e2ab7

Co-Authored-By: mish@e2b.dev <mish@e2b.dev>
@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment, CI, and merge conflict monitoring

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM — trivial, low-risk addition of a single alias key to an existing lookup map.

Extended reasoning...

Overview

The change adds one line to TAB_URL_MAP in src/configs/dashboard-tab-url-map.ts, mapping the new project tab key to the same PROTECTED_URLS.GENERAL(teamSlug) destination already used by team, settings, and general. No other files are touched.

Security risks

None. This is a static, hardcoded URL-builder map with no user input influencing the mapping keys beyond a simple lookup, and the added entry points to an already-existing, already-reachable URL builder. It does not introduce new routes, permissions, or data exposure.

Level of scrutiny

Minimal scrutiny is warranted. This is a config-style, additive, backwards-compatible change following an exact existing pattern in the same file (three other keys already map to the identical destination). The PR description also notes the existing dynamic integration test in dashboard-route.test.ts iterates over TAB_URL_MAP, so the new key is automatically covered.

Other factors

No CODEOWNERS or security-sensitive paths are involved, there are no outstanding review comments to address, and the diff is a single line. This is exactly the kind of mechanical, low-ambiguity change that does not need human review.

@mishushakov
mishushakov merged commit 9933b98 into private/main Jul 24, 2026
3 checks passed
@mishushakov
mishushakov deleted the devin/1784840497-tab-project-alias branch July 24, 2026 14:03
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