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: upgreade the virtual-event demo app to stream chat react v10 #68

Merged
merged 43 commits into from
Nov 3, 2022
Merged

feat: upgreade the virtual-event demo app to stream chat react v10 #68

merged 43 commits into from
Nov 3, 2022

Conversation

MartinCupela
Copy link
Contributor

🎯 Goal

Make use of the new theming V2 updates in the stream-chat-react SDK:

  • Upgrade to the latest version of stream-chat-react (v10.3.0)
  • Convert the CSS rules to use the theming V2 variables system
  • Remove redundant CSS rules
  • Remove redundant custom components
  • Fix bugs:
    • send reply as a direct message
    • thread re-rendering on every message post
    • visual bugs (overflows, broken layouts, ...)

@MartinCupela
chore(virtual-event): upgrade dependencies
8bc52bb
@MartinCupela
chore(virtual-event): adjust component props types for React 18, fix …
45443ff 
…StreamGeneric type
@MartinCupela
refactor(virtual-event): do not use deprecated Chat prop customStyles
4ffd0dd
@MartinCupela
refactor(virtual-event): adapt channel search to its new controller API
16ae9e3
@MartinCupela
refactor(virtual-event): activate theming v2
8a0fe74
@MartinCupela
refactor(virtual-event): adapt channel search to theming v2
aa85c6f
@MartinCupela
fix(virtual-event): prevent message author name overflow
b4c27e2
@MartinCupela
fix(virtual-event): adapt DMChannelPreview height to its content
67e0c25
@MartinCupela
refactor(ChatHeader): remove repetitive code
852a981
@MartinCupela
feat: render app with React 18 createRoot
31de16d
@MartinCupela
chore: gitignore .idea in website-react-examples
0d58bd9
@MartinCupela
chore(deps): bump stream-chat-react to 10.3.0
7150285
@MartinCupela
feat: override stream-chat-css variables
815ce6c
@MartinCupela
feat: add global variables
1638a27
@MartinCupela
fix: remove message components bugs, distribute the css into separate…
a1cc260 
…s stylesheets
@MartinCupela
fix: memoize submit handler
dc43094
@MartinCupela
fix: remove unnecessary search code, distribute search related styles…
e6a0096 
… into stylesheets
@MartinCupela
fix: adjust message input and distribute styles into separate files
67b8646
@MartinCupela
fix: adjust thread code and distribute styles into separate files
bc2f0d2
@MartinCupela
fix: tooltip word wrapping
6bb09cf
@MartinCupela
refactor: manage dropdown state with useBoolState hook in Participant…
98ea49c 
…Profile
@MartinCupela
refactor: extract new component ChannelHeader
f209d22
@MartinCupela
fix: remove style bugs and separate chat component styles into layout…
98ad3f2 
… and theme sheets
@MartinCupela
types: remove generic from ChannelPreviewUIComponentProps
ff7321d
@MartinCupela
refactor: remove UserEllipse component, export UpvoteThumb
59889c6
@MartinCupela
fix: show scroll bars on RoomVideo only when necessary
226291d
@MartinCupela
fix: extend the room video background height
53c03ff
@MartinCupela
fix: fallback to user avatar on image load failure
58505fb
@MartinCupela
style: fix lint issues
2b23857
@MartinCupela
chore: install react-scripts as dev dep
c57a670
@MartinCupela MartinCupela marked this pull request as ready for review October 20, 2022 13:25
@MartinCupela MartinCupela merged commit 799affa into master Nov 3, 2022
@MartinCupela MartinCupela deleted the virtual-event-upgrade-to-stream-chat-react-v10 branch November 3, 2022 13:46
@MartinCupela MartinCupela mentioned this pull request Nov 9, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@oliverlaz oliverlaz oliverlaz approved these changes

@petyosi petyosi Awaiting requested review from petyosi

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

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

Successfully merging this pull request may close these issues.

None yet
2 participants
@MartinCupela @oliverlaz