feat: add COSCUP 2026 event landing page - #740
Conversation
Add the COSCUP 2026 event with talk details, and extend the EventLanding component to display start/end time and room. Signed-off-by: Reza Jelveh <fishmangit@dynamia.ai>
hami-robot
Bot
added
kind/documentation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: fishman The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
✅ Deploy Preview for project-hami ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
📝 WalkthroughWalkthroughAdds COSCUP 2026 event data and a landing page. ChangesCOSCUP 2026 landing flow
Estimated code review effort: 2 (Simple) | ~10 minutes Sequence Diagram(s)sequenceDiagram
participant Browser
participant LandingPage as coscup-2026.js
participant EventLanding
participant EventsData as events.js
Browser->>LandingPage: request COSCUP 2026 landing page
LandingPage->>EventLanding: render with slug coscup-2026
EventLanding->>EventsData: load coscup-2026 event
EventsData-->>EventLanding: return event metadata
EventLanding-->>Browser: render event page
Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@src/data/events.js`:
- Around line 114-118: Update the event record’s location object near room
"TR214" to include the official venue, National Taiwan University of Science and
Technology, and its address in both localized values. Keep the existing room
code unchanged so EventLanding renders an actionable venue and room combination.
🪄 Autofix
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro Plus
Run ID: b3511e96-e519-464e-b72a-a869ccd4cf93
📒 Files selected for processing (3)
src/components/EventLanding.jssrc/data/events.jssrc/pages/landing/coscup-2026.js
| location: { | ||
| en: "Taipei, Taiwan", | ||
| zh: "中国台湾 台北", | ||
| }, | ||
| room: "TR214", |
There was a problem hiding this comment.
🗄️ Data Integrity & Integration | 🟡 Minor | ⚡ Quick win
Include the venue with the room code.
This record sets event.location to only Taipei, Taiwan, while EventLanding renders that value beside event.room. The result is Taipei, Taiwan · TR214; TR214 is not actionable without the building. Add the COSCUP venue and address so attendees can locate the room and the existing structured-data location remains useful. The official event and session pages identify National Taiwan University of Science and Technology as the venue. (coscup.org)
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@src/data/events.js` around lines 114 - 118, Update the event record’s
location object near room "TR214" to include the official venue, National Taiwan
University of Science and Technology, and its address in both localized values.
Keep the existing room code unchanged so EventLanding renders an actionable
venue and room combination.
add coscup slide deck Signed-off-by: Reza Jelveh <fishmangit@dynamia.ai>
2 participants
Add the COSCUP 2026 event with talk details, and extend the
EventLanding component to display start/end time and room.
Signed-off-by: Reza Jelveh fishmangit@dynamia.ai
What type of PR is this?
/kind documentation
What this PR does / why we need it:
CosCup landing
Summary by CodeRabbit