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

Fix: Solve the problem of adding new conversations and automatically updating the conversation list #1806

Closed
wants to merge 4 commits into from
Closed

Fix: Solve the problem of adding new conversations and automatically updating the conversation list #1806

wants to merge 4 commits into from

Conversation

dusx1981
Copy link

@dusx1981 dusx1981 commented Aug 10, 2024 •
edited by Aries-ckt
Loading

尝试解决 Close #1762

@dusx1981 dusx1981 marked this pull request as draft August 10, 2024 01:55
@dusx1981 dusx1981 marked this pull request as ready for review August 10, 2024 01:55
Aries-ckt
Aries-ckt approved these changes Aug 10, 2024
View reviewed changes
Copy link
Collaborator

@Aries-ckt Aries-ckt left a comment

Choose a reason for hiding this comment

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

LGTM.

@Aries-ckt Aries-ckt changed the title 解决添加新的对话，对话列表自动更新问题 Fix: Solve the problem of adding new conversations and automatically updating the conversation list Aug 10, 2024
fangyinc
fangyinc requested changes Aug 20, 2024
View reviewed changes
Copy link
Collaborator

@fangyinc fangyinc left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Hi @dusx1981 , thank you for your contribution, but after my test I found some minor problems.

A large number of repeated requests will be generated during the conversation.

image

@dusx1981 dusx1981 marked this pull request as draft August 20, 2024 07:11
@dusx1981
Copy link
Author

dusx1981 commented Aug 22, 2024 •
edited
Loading

Hi @dusx1981 , thank you for your contribution, but after my test I found some minor problems.

A large number of repeated requests will be generated during the conversation.

image

@fangyinc 因为一次对话有多轮的前后台交互，新建的聊天记录，在第一次对话最终完成后，才会被后台加入列表，更新才能有效，所以本地代码设计了两阶段刷新，解决频繁请求列表的问题，等主分支可以编译了，我再提个更新的 PR

@dusx1981 dusx1981 closed this Aug 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fangyinc fangyinc fangyinc requested changes

@Aries-ckt Aries-ckt Aries-ckt approved these changes

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

Successfully merging this pull request may close these issues.

[Bug] 聊天历史更新问题
3 participants
@dusx1981 @Aries-ckt @fangyinc