-
Notifications
You must be signed in to change notification settings - Fork 468
✨ Add a Featured Agents section to the agent market #2210
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
base: develop
Are you sure you want to change the base?
Conversation
feat:修复,补充类型定义 + 给隐式 any 的参数加类型
7f69eed to
3ad8574
Compare
frontend/types/market.ts
Outdated
| // Request featured items in the response (optional) | ||
| include_featured?: boolean; | ||
| // Limit how many featured items to return; undefined means no limit | ||
| featured_limit?: number; |
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.
这两个参数应该已经删除了吧
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.
是的,已删除,感谢检视意见
| const defaultClasses = | ||
| "bg-white dark:bg-slate-800 border-slate-200 dark:border-slate-700 hover:border-purple-300 dark:hover:border-purple-600 hover:shadow-lg"; | ||
| const featuredClasses = | ||
| "bg-white dark:bg-slate-800 border-slate-200 dark:border-slate-700 text-slate-900 dark:text-white hover:border-purple-300 dark:hover:border-purple-600 hover:shadow-lg"; |
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.
以上三种样式并不存在复用,不需要单独抽取为常量
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.
已修正,请再次检视
#2203





✨ 智能体市场增加精选栏,英文版本同步上线。已测试过某页签仅精选、仅普通、精选无需翻页时灰色拖拽条隐藏等功能。