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

Updated Stream Chat #624

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Updated Stream Chat #624

wants to merge 3 commits into from

Conversation

dzwood
Copy link
Contributor

@dzwood dzwood commented Oct 19, 2023

About

This PR updates the following packages: stream-chat, stream-chat-react. It removes deprecated components from the new update and adds the correct ones. We may want to revisit the MessageInput component and create our own custom one: https://getstream.io/chat/docs/sdk/react/guides/theming/input_ui/ later on. But what they have out of the box should work great for now.

Test Instructions

  • Log in and go to a group
  • Test the updated chat UI

Note: the "SendButton" does not show in desktop(mobile only).

Screenshots

image

Closes Tickets

CFDP-2736

* We may want to revisit the `MessageInput` component and create our own custom one: https://getstream.io/chat/docs/sdk/react/guides/theming/input_ui/
@vercel
Copy link

vercel bot commented Oct 19, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
web-app-v2 ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 30, 2023 8:28pm

@dzwood dzwood self-assigned this Oct 19, 2023
@dzwood dzwood added Enhancement ✨ New feature or request Ready for Review ✅ PR is Ready for Review labels Oct 19, 2023
leoalfonso14
leoalfonso14 previously approved these changes Oct 20, 2023
Copy link
Contributor

@leoalfonso14 leoalfonso14 left a comment

Choose a reason for hiding this comment

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

Works great, the send icon for desktop does show up once you type something.

Copy link
Contributor

@codingwithlis codingwithlis left a comment

Choose a reason for hiding this comment

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

Looks great, reacting to other's messages is being inconsistent, sometimes I can react sometimes I can't

@dzwood
Copy link
Contributor Author

dzwood commented Oct 30, 2023

@leoalfonso14 @codingwithlis Sorry I forgot to import the new CSS files for v2🙃, it should look much better now🎨

Copy link
Contributor

@leoalfonso14 leoalfonso14 left a comment

Choose a reason for hiding this comment

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

Awesome job!

@dzwood dzwood added 🛑 DO NOT MERGE 🛑 and removed Ready for Review ✅ PR is Ready for Review labels Mar 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants