Skip to content

Conversation

@Weixuanf
Copy link
Member

… the original editorContent or not and to overwrite that if the user wants overwriting to happen

… the original editorContent or not and to overwrite that if the user wants overwriting to happen
Comment on lines 284 to 286
// model: model.model,
// provider: model.provider,
// url: model.url,
Copy link
Member Author

Choose a reason for hiding this comment

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

这里为啥都comment掉了?如果没有这些 发到server没法更换模型请求吧?

Copy link
Collaborator

Choose a reason for hiding this comment

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

这些当时就是为了test,就后端返回的也没用AI

# create new session
prompt = messages[0].get('content', '')
await db_service.create_chat_session(session_id, model, provider, (prompt[:200] if isinstance(prompt, str) else ''))
# data = await request.json()
Copy link
Member Author

Choose a reason for hiding this comment

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

这里都删掉了是测试用的吗

Copy link
Collaborator

Choose a reason for hiding this comment

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

对对对

};
// const setEditorContentWrapper = (content: string) => {
// setEditorContent(content);
// debouncedUpdateFile(content);
Copy link
Member Author

Choose a reason for hiding this comment

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

这个debounceUpdateFile()之后还是得加回来吧 这个是保存到磁盘上的file的

Copy link
Collaborator

@stevenxichengao stevenxichengao May 26, 2025

Choose a reason for hiding this comment

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

那个setEditorContentWrapper我放到parent component里面了因为ChatInterface也需要,现在是个prop

@Weixuanf Weixuanf force-pushed the main branch 5 times, most recently from d6c5843 to 9a08aa5 Compare May 29, 2025 12:19
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