Skip to content

Conversation

@ycdzj
Copy link
Contributor

@ycdzj ycdzj commented Oct 24, 2025

No description provided.

Copilot AI review requested due to automatic review settings October 24, 2025 08:44
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes the member count display for namespaces by changing the API endpoint from fetching the full member list to fetching just the count, improving performance and data handling.

  • Updated the API endpoint from namespaces/${namespaceId}/members to namespaces/${namespaceId}/members/count
  • Changed the data structure from an array of members to an object containing a count property
  • Updated the UI component to access the count property instead of using array length

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
src/hooks/use-namespace-member.ts Changed API endpoint to fetch member count instead of full member list and updated state type accordingly
src/page/sidebar/switcher/member.tsx Updated to access count property from data object instead of array length

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@LucienShui LucienShui merged commit 0b2b3d9 into main Oct 24, 2025
2 checks passed
@LucienShui LucienShui deleted the refactor/namespaces branch October 24, 2025 15:11
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.

3 participants