Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Implemented search member and display member info feature #708

Merged
merged 10 commits into from
Dec 22, 2024

Conversation

abirc8010
Copy link
Contributor

@abirc8010 abirc8010 commented Dec 21, 2024

Brief Title

Implemented a search and display member info feature that allows searching members and showing detailed information on clicking a particular member.

Acceptance Criteria fulfillment

  • Included a search box in the 'Members' section.
  • The search updates the member list dynamically as the user types using members.filter based on username and name attribute
  • Utilized useSetExclusiveState and useUserStore to display the userInfo by attaching an onClick event inside RoomMemberItem

Fixes #707

Video/Screenshots

Screencast.from.2024-12-21.22-15-54.webm

Note: The PR will be ready for live testing at https://rocketchat.github.io/EmbeddedChat/pulls/pr-708 after approval. Contributors are requested to replace <pr_number> with the actual PR number.

Copy link
Collaborator

@Spiral-Memory Spiral-Memory left a comment

Choose a reason for hiding this comment

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

LGTM

@Spiral-Memory
Copy link
Collaborator

Hi @abirc8010

Looks good to me. Great work!

However, the width of the user info side panel is not the same as the others. Could you please fix that in this PR? Also, please add the required padding to the other side panels, as I mentioned in the last merged PR.

@Spiral-Memory
Copy link
Collaborator

Let me know once you fix both

@abirc8010
Copy link
Contributor Author

Hey @Spiral-Memory I have fixed the issues.

@Spiral-Memory Spiral-Memory merged commit 94e0faf into RocketChat:develop Dec 22, 2024
4 checks passed
@Spiral-Memory
Copy link
Collaborator

Great work @abirc8010

github-actions bot added a commit that referenced this pull request Dec 22, 2024
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.

Feat: Include 'search member' and 'display member info' feature
2 participants