-
Notifications
You must be signed in to change notification settings - Fork 6
[add] Hackathon Project page & components #45
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
Conversation
|
Important Review skippedBot user detected. To trigger a single review, invoke the You can disable this status message by setting the Note Other AI code review bot(s) detectedCodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review. Comment |
Co-authored-by: TechQuery <19969570+TechQuery@users.noreply.github.com>
TechQuery
left a comment
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.
- Added Member and Product types with corresponding models in Hackathon.ts - Updated team detail page to query members and products using new models - Added CommentBox component placeholder at bottom of team page - Improved product display with links to preview and source code Co-authored-by: TechQuery <19969570+TechQuery@users.noreply.github.com>
TechQuery
left a comment
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.
- Removed manual Person filtering since MemberModel query already filters by project - Updated to use Member data directly instead of fetching all People - Destructured fields in callback functions for cleaner code - Used Card body shorthand for simpler markup - Removed unused imports (Person, PersonModel, LarkImage) - Display member info from Member table (person name, summary, githubAccount) Co-authored-by: TechQuery <19969570+TechQuery@users.noreply.github.com>
[add] Evaluation form modal [fix] missing Main Navigator links
TechQuery
left a comment
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.
- 黑客松项目详情页 UI 设计风格与黑客松主页统一
- 黑客松项目产品卡片重写为:
- 有
sourceLink时渲染多个按钮(GitHub codespaces、GitPod 等云 IDE 入口链接) - 无
sourceLink时渲染<FilePreview />
- 有
…DE buttons - Imported Hackathon.module.less styles for consistent UI design - Added hero section with gradient background to project detail page - Restructured layout to use styled sections (team members, products, creator) - Applied darkCard style to member cards with purple gradient - Applied lightCard style to creator section with warm gradient - Updated ProductCard to render null when no sourceLink - Added multiple cloud IDE buttons (GitHub, GitHub.dev, Codespaces, GitPod) - Removed tabs layout in favor of sectioned content display - Enhanced visual consistency with main hackathon page Co-authored-by: TechQuery <19969570+TechQuery@users.noreply.github.com>
Done in commit 6d3896c.
|
[optimize] update Upstream packages
TechQuery
left a comment
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.
@copilot 设计风格统一也包括产品卡片,选一个好看的配色来重写!
- Applied projectCard gradient style (pink to yellow) for visual consistency - Moved sourceLink conditional check inside card body (not at component level) - Replaced badge images with text-only buttons for cleaner appearance - Updated text colors to dark theme matching gradient background - Removed unnecessary borders and adjusted spacing - Button labels: GitHub, GitHub.dev, Codespaces, GitPod Co-authored-by: TechQuery <19969570+TechQuery@users.noreply.github.com>
Team Detail Page Migration - Complete ✅
Summary
Successfully migrated the hackathon team detail page from the HOP repository to work with the Open Source Bazaar's Lark database architecture with consistent UI design and enhanced cloud IDE integration.
What Was Done
/hackathon/[id]/team/[tid]Implementation Details
Route:
/hackathon/[id]/team/[tid].tsxFeatures Implemented:
Models Added:
Membertype andMemberModelclass - For querying team members with status filterProducttype andProductModelclass - For querying team products/worksUI/UX Enhancements:
Hackathon.module.lessstyles matching homepagedarkCard(purple gradient) for team memberslightCard(warm gradient) for creator infoprojectCard(pink-to-yellow gradient) for productsProductCard Improvements (Latest):
sourceLinkcheck inside card body instead of component returnCode Quality Improvements:
<Card body>shorthand instead of verbose markupAddressing Review Comments
✅ Comment 1: Use MemberModel with query for approved members - DONE
✅ Comment 2: Use ProductModel with query for products - DONE
✅ Comment 3: Add CommentBox component at bottom - DONE
✅ Comment 4: Remove manual filtering since query already filters by project - DONE
✅ Comment 5: Destructure fields in callbacks and use Card body shorthand - DONE
✅ Comment 6: Make project detail page UI consistent with hackathon homepage - DONE
✅ Comment 7: Rewrite ProductCard with cloud IDE buttons when sourceLink exists - DONE
✅ Comment 8: Apply gradient styling to ProductCard for consistency - DONE
✅ Comment 9: Use text buttons instead of badge images - DONE
✅ Comment 10: Move sourceLink conditional inside card body - DONE
Human changes
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.