Drop "Admin Dev" sidebar option

[oscartbeaumont]

It points to a 404 page now.

Summary by CodeRabbit

• Refactor
  • Streamlined dashboard navigation by removing the “Admin Dev” menu item.
  • Users who previously saw this option will now have a simplified, consistent menu across environments.
  • No changes to other navigation items or behaviors.

[coderabbitai]

Walkthrough

The conditional injection of the "Admin Dev" navigation item in AdminNavItems was removed from Items.tsx. Navigation is now generated without checking buildEnv.NEXT_PUBLIC_IS_CAP or user.email domain. No other logic, exports, or error handling were changed.

Changes

Cohort / File(s)	Summary
Dashboard Navbar items apps/web/app/(org)/dashboard/_components/Navbar/Items.tsx	Removed conditional creation/injection of the "Admin Dev" nav item tied to buildEnv.NEXT_PUBLIC_IS_CAP and user.email ending with "@cap.so". Navigation generation now excludes this dynamic entry. No other code paths altered.

Sequence Diagram(s)

sequenceDiagram
  autonumber
  actor User
  participant Navbar as Navbar Component
  participant AdminNav as AdminNavItems

  User->>Navbar: Open dashboard
  Navbar->>AdminNav: Build navigation items
  rect rgba(200,220,255,0.25)
    note over AdminNav: Old behavior
    AdminNav-->>AdminNav: If NEXT_PUBLIC_IS_CAP && user.email endsWith("@cap.so")<br/>append "Admin Dev"
  end
  rect rgba(200,255,200,0.25)
    note over AdminNav: New behavior
    AdminNav-->>Navbar: Return static items (no conditional "Admin Dev")
  end
  Navbar-->>User: Render navbar items

Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

I hop through menus, neat and deft,
One secret tab has now been left.
No more cap-so whisper in the breeze,
The nav is calm, simplified with ease.
Thump-thump—clean paths, fewer bends,
A tidy warren for all my friends. 🐇✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	The title succinctly and accurately describes the main change of removing the "Admin Dev" sidebar option, matching the code modifications to drop that navigation item and clearly conveying the purpose of the pull request.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch drop-admin-dev-sidebar-option

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 99770b9 and 0b993db.

📒 Files selected for processing (1)

• apps/web/app/(org)/dashboard/_components/Navbar/Items.tsx (0 hunks)

💤 Files with no reviewable changes (1)

• apps/web/app/(org)/dashboard/_components/Navbar/Items.tsx

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)

• GitHub Check: Build Desktop (aarch64-apple-darwin, macos-latest)
• GitHub Check: Build Desktop (x86_64-pc-windows-msvc, windows-latest)
• GitHub Check: Analyze (rust)

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}