Skip to content

UI fixes#478

Merged
Benjtalkshow merged 23 commits intoboundlessfi:mainfrom
Benjtalkshow:ui-fixes
Mar 10, 2026
Merged

UI fixes#478
Benjtalkshow merged 23 commits intoboundlessfi:mainfrom
Benjtalkshow:ui-fixes

Conversation

@Benjtalkshow
Copy link
Collaborator

@Benjtalkshow Benjtalkshow commented Mar 10, 2026

Summary by CodeRabbit

  • New Features
    • Projects count badge now displays in sidebar navigation when greater than zero
    • Profile now consolidates and tracks total projects created across all sources

@vercel
Copy link

vercel bot commented Mar 10, 2026

@Benjtalkshow is attempting to deploy a commit to the Threadflow Team on Vercel.

A member of the Team first needs to authorize it.

@Benjtalkshow Benjtalkshow merged commit 12a5ccd into boundlessfi:main Mar 10, 2026
4 of 6 checks passed
@coderabbitai
Copy link

coderabbitai bot commented Mar 10, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 1243cb46-d9af-4e26-88e2-62338315f70a

📥 Commits

Reviewing files that changed from the base of the PR and between 2f7335f and cfd9d54.

📒 Files selected for processing (3)
  • app/me/layout.tsx
  • components/app-sidebar.tsx
  • components/landing-page/navbar.tsx

Disabled knowledge base sources:

  • Linear integration is disabled

You can enable these sources in your CodeRabbit configuration.


📝 Walkthrough

Walkthrough

The changes extend the application's profile data structure to track created projects, introduce computed project count logic in the layout component, and wire this count through the sidebar's navigation system to display a dynamic projects badge. Additionally, the Logo component is exported for reuse across the codebase.

Changes

Cohort / File(s) Summary
Profile Data Structure Extension
app/me/layout.tsx
Extended MeLayoutProfile interface with user.projects, root-level projects, and stats.projectsCreated fields; introduced memoized projectsCount computation that prioritizes stats.projectsCreated or deduplicates merged project arrays by id.
Sidebar Navigation Integration
components/app-sidebar.tsx
Extended AppSidebar props and getNavigationData() function signature to accept projects count; added dynamic badge display for Projects navigation item based on counts.projects value (shown when > 0).
Component Export
components/landing-page/navbar.tsx
Converted two Logo function definitions from non-exported to exported functions for reusability across the application.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

  • PR #430: Modifies the same files (app/me/layout.tsx and components/app-sidebar.tsx) to extend sidebar counts and navigation data structure in a similar pattern (submissions count addition).

Suggested reviewers

  • 0xdevcollins

Poem

🐰 Hop! Hop! Projects now counted with care,
Badges bloom bright in the sidebar's fair square,
Data flows fresh through the layout's designed,
Logo exported, shared joy we shall find!

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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

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

@coderabbitai coderabbitai bot mentioned this pull request Mar 10, 2026
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