-
Notifications
You must be signed in to change notification settings - Fork 55
Add support for comment websocket messages #5577
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
base: master
Are you sure you want to change the base?
Conversation
eikhr
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
|
Is there a backend PR? |
See webkom/lego#3835 |
Arashfa0301
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work Isak
|
Dont know if all the redux things is correct. I keep getting "An input selector returned a different result when passed same arguments" as warning. I also created a component that handles all the ws group logic. It passes on status and even a nice little component indicating the status. |
e1ff38d to
8b5fafb
Compare
ch0rizo
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Huge 🪨
jonasdeluna
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀
|
get this in, please! |
524e02f to
0d90c0c
Compare
0d90c0c to
2a2eb09
Compare
Description
Make it possible for comments to be updated live using websockets.
Backend PR for sending ws messages in progress
Result
Using different browsers:
Screen.Recording.2025-05-12.at.12.14.05.mov
Testing