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

设置项未修改也显示有变动 #16

Closed
hotip opened this issue Aug 7, 2023 · 0 comments
Closed

设置项未修改也显示有变动 #16

hotip opened this issue Aug 7, 2023 · 0 comments

Comments

@hotip
Copy link
Contributor

hotip commented Aug 7, 2023

isModified 方法有误
@hotip hotip mentioned this issue Aug 7, 2023
Merged
@hotip hotip closed this as completed Aug 7, 2023
phodal added a commit that referenced this issue Mar 5, 2024
@phodal
refactor(SSE handling): improve SSE parsing logic #97
3188b9e 
Refactored the Server Sent Event (SSE) parsing logic to handle edge cases better and make code cleaner. The new approach fixes #16 which involved adding checks on whether incoming events were `"ping":` or just starting with `":"` followed by "ping". It also added functionality to clear chat window before showing new messages while creating a new conversation using NewChatAction component. Made this change because it enhances user experience by providing smoother interaction between server push notifications and client display of those notifications. Modified these files accordingly as per suggestions from SonarLint analysis.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@hotip