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: resolve the display issue on the bot configuration page. #549

Merged
merged 3 commits into from
Dec 2, 2024

Conversation

xingwanying
Copy link
Contributor

@xingwanying xingwanying commented Dec 2, 2024

  • 修复使用默认token_id传参错误问题
  • fix typo
  • 修复编辑页面llm配置显示问题

image

close #544
close #487

Copy link

vercel bot commented Dec 2, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
petercat ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 2, 2024 6:22am

Copy link

Walkthrough

此PR修复了在bot配置页面上显示问题,主要涉及到默认token_id传参错误和编辑页面llm配置显示问题。通过更正拼写错误和更新逻辑来确保正确显示。

Changes

文件 概要
client/app/factory/edit/components/BotCreateForm.tsx 修正了拼写错误并更新了默认选择逻辑。
client/app/factory/edit/page.tsx 更新了gitUrl的默认值逻辑。
client/app/hooks/useAvailableLLMs.ts 新增了useAvailableLLMs钩子。
client/app/hooks/useAvaliableLLMs.ts 移除了旧的useAvaliableLLMs钩子。
client/app/services/UserController.ts 修正了getAvailableLLMs函数名称。
server/bot/router.py 更新了数据库查询字段以包含llm和token_id。

Copy link

codecov bot commented Dec 2, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Files with missing lines Coverage Δ
server/bot/router.py 18.13% <ø> (-1.14%) ⬇️

... and 2 files with indirect coverage changes

@xingwanying xingwanying merged commit c20a74c into main Dec 2, 2024
5 checks passed
@xingwanying xingwanying deleted the fix_edit branch December 2, 2024 06:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant