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: render user list for new chat modal #474

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

ice-kreios
Copy link
Contributor

@ice-kreios ice-kreios commented Dec 26, 2024

Description

  • Implemented two list views with backend integration: Following users list (default view), Searched users list with pagination

Additional Notes

  • Implemented dynamic chat relay management that mirrors user's main relays

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Refactoring
  • Documentation
  • Chore

Screenshots (if applicable)

image

@ice-kreios ice-kreios force-pushed the feature/new-chat-user-list branch from 68f3114 to 4e9ce44 Compare December 27, 2024 15:08
@ice-kreios ice-kreios marked this pull request as ready for review December 27, 2024 15:09
@ice-kreios ice-kreios requested review from a team as code owners December 27, 2024 15:09
@ice-kreios ice-kreios force-pushed the feature/new-chat-user-list branch 2 times, most recently from d2cb841 to ea02918 Compare December 28, 2024 18:52
@ice-kreios ice-kreios enabled auto-merge (squash) December 28, 2024 18:53
@ice-kreios ice-kreios force-pushed the feature/new-chat-user-list branch from ea02918 to 1ad863b Compare December 30, 2024 07:57
ice-alcides
ice-alcides previously approved these changes Dec 30, 2024

@riverpod
List<EntitiesDataSource> usersDataSource(Ref ref) {
final searchText = ref.watch(usersSearchTextProvider);
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we need a debounce or a throttle?

@ice-kreios ice-kreios force-pushed the feature/new-chat-user-list branch from 826d209 to 3023e2a Compare December 31, 2024 16:05
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