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.

Sign up for GitHub

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

Jump to bottom

feat: reactive channel list poc #2924

Draft
wants to merge 26 commits into
base: develop
Choose a base branch
from
Draft

feat: reactive channel list poc #2924

wants to merge 26 commits into from

Conversation

isekovanic
Copy link
Contributor

@isekovanic isekovanic commented Jan 31, 2025 •
edited
Loading

🎯 Goal

An SDK PR for this change in the LLC.

TODO:

  • Bump the stream-chat version once it's been released

🛠 Implementation details

🎨 UI Changes

iOS
Before After
Android
Before After

🧪 Testing

☑️ Checklist

  • I have signed the Stream CLA (required)
  • PR targets the develop branch
  • Documentation is updated
  • New code is tested in main example apps, including all possible scenarios
    • SampleApp iOS and Android
    • Expo iOS and Android

@isekovanic
feat: implement channel manager poc
fe44060
@isekovanic
chore: use reactive state
63895f2
@isekovanic
fix: properly cover rerendering issues
22f7e6c
@isekovanic
fix: wrap creation in separate callback
3b3d4e6
@isekovanic
chore: add event handler overrides
44c4cba
@isekovanic
fix: properly resolve loadingChannels
c1a78d2
@isekovanic
fix: keep user presence handler in sdk
4d073c1
@isekovanic
fix: use channel list ready state
10ad331
@isekovanic
fix: move state to memo
b856437
@isekovanic
Merge branch 'develop' into feat/reactive-channel-list-poc
2225971 
# Conflicts:
#	package/src/components/ChannelList/ChannelList.tsx
@isekovanic
fix: merge issues
c9ee5c4
@isekovanic
fix: properly override member.updated
dbb534d
@isekovanic
chore: notification.added_to_channel arch/pinning changes
90f7cef
@isekovanic
chore: add support for notification.message_new
2407b9c
@isekovanic
Merge branch 'develop' into feat/reactive-channel-list-poc
80b5f7f
@isekovanic
fix: broken loading state on slower queryChannels
966387f
@isekovanic
fix: return channel.updated to SDK and fix stale query check
87d11d0
@isekovanic
Merge branch 'develop' into feat/reactive-channel-list-poc
e771d52
@isekovanic
fix: broken error state in query backoff
31eeadc
@isekovanic
chore: add explanation
71dbb5f
@isekovanic
fix: move the setting of overrides in separate hook
70bfe59
@isekovanic
fix: add default for breaking prop
621d116
@isekovanic
fix: move default down to llc
16329c5
@isekovanic isekovanic mentioned this pull request Jan 31, 2025
Draft
4 tasks
@isekovanic isekovanic marked this pull request as draft January 31, 2025 16:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@oliverlaz oliverlaz Awaiting requested review from oliverlaz

@myandrienko myandrienko Awaiting requested review from myandrienko

@szuperaz szuperaz Awaiting requested review from szuperaz

@MartinCupela MartinCupela Awaiting requested review from MartinCupela

@khushal87 khushal87 Awaiting requested review from khushal87

@arnautov-anton arnautov-anton Awaiting requested review from arnautov-anton

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@isekovanic