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

fix: enable enableInjectSystemPrompts attribute for old sessions #2392

Merged
merged 2 commits into from
Jul 16, 2023

Conversation

imldy
Copy link
Contributor

@imldy imldy commented Jul 14, 2023

Bug introduced by fix #2344

@imldy
Copy link
Contributor Author

imldy commented Jul 14, 2023

或许这样更合适:在不存在enableInjectSystemPrompts属性的旧会话提问时,获取全局配置中的值并设为当前会话的值。

因为打开会话可能只是看看记录,而发起提问则代表正式使用当前会话配置,没有此属性时使用全局配置较为合理。

@Issues-translate-bot
Copy link

Bot detected the issue body's language is not English, translate it automatically.


Perhaps it is more appropriate to get the value in the global configuration when opening an old session where the enableInjectSystemPrompts property does not exist

@Yidadaa Yidadaa merged commit 7073fd2 into ChatGPTNextWeb:main Jul 16, 2023
alchemist139 pushed a commit to alchemist139/ChatGPT-Next-Web that referenced this pull request Sep 21, 2023
…Prompts

fix: enable `enableInjectSystemPrompts` attribute for old sessions
gaogao1030 pushed a commit to gaogao1030/ChatGPT-Next-Web that referenced this pull request May 16, 2024
…Prompts

fix: enable `enableInjectSystemPrompts` attribute for old sessions
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants