-
-
Notifications
You must be signed in to change notification settings - Fork 11.2k
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: fix modelList
merge
#4491
Conversation
@hezhijie0327 is attempting to deploy a commit to the LobeChat Community Team on Vercel. A member of the Team first needs to authorize it. |
Thank you for raising your pull request and contributing to our Community |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #4491 +/- ##
==========================================
- Coverage 92.30% 92.29% -0.01%
==========================================
Files 496 496
Lines 36207 36194 -13
Branches 3440 3436 -4
==========================================
- Hits 33420 33407 -13
Misses 2787 2787
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
其实这样更精简吧
Object.values(ModelProvider).forEach((id) =>{
const provider = draft.find((d) => d.id === id);
if (provider) provider.chatModels = mergeModels(id as any, provider.chatModels);
})
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
改好了 |
Changed |
❤️ Great PR @hezhijie0327 ❤️ The growth of project is inseparable from user feedback and contribution, thanks for your contribution! If you are interesting with the lobehub developer community, please join our discord and then dm @arvinxx or @canisminor1990. They will invite you to our private developer channel. We are talking about the lobe-chat development or sharing ai newsletter around the world. |
### [Version 1.25.1](v1.25.0...v1.25.1) <sup>Released on **2024-10-26**</sup> #### 🐛 Bug Fixes - **misc**: Fix `modelList` merge. <br/> <details> <summary><kbd>Improvements and Fixes</kbd></summary> #### What's fixed * **misc**: Fix `modelList` merge, closes [#4491](#4491) ([ea422d8](ea422d8)) </details> <div align="right"> [![](https://img.shields.io/badge/-BACK_TO_TOP-151515?style=flat-square)](#readme-top) </div>
🎉 This PR is included in version 1.25.1 🎉 The release is available on: Your semantic-release bot 📦🚀 |
### [Version 1.68.1](v1.68.0...v1.68.1) <sup>Released on **2024-10-26**</sup> #### 🐛 Bug Fixes - **misc**: Fix `modelList` merge. <br/> <details> <summary><kbd>Improvements and Fixes</kbd></summary> #### What's fixed * **misc**: Fix `modelList` merge, closes [lobehub#4491](https://github.com/bentwnghk/lobe-chat/issues/4491) ([ea422d8](ea422d8)) </details> <div align="right"> [![](https://img.shields.io/badge/-BACK_TO_TOP-151515?style=flat-square)](#readme-top) </div>
💻 变更类型 | Change Type
🔀 变更说明 | Description of Change
close #4485
📝 补充信息 | Additional Information
Before
After
标签测试