Skip to content

feat: add Our Department section for About Us page#349

Open
khangronky wants to merge 15 commits into
mainfrom
feat/our-department-section
Open

feat: add Our Department section for About Us page#349
khangronky wants to merge 15 commits into
mainfrom
feat/our-department-section

Conversation

@khangronky

Copy link
Copy Markdown
Collaborator

Description

What?

Why?

How?

Screenshots for proof (must have)

@khangronky khangronky marked this pull request as draft March 23, 2026 17:43
@khangronky khangronky self-assigned this Mar 23, 2026
@khangronky khangronky added the WIP Work in progress - NOT READY TO BE MERGED label Mar 23, 2026
@khangronky khangronky marked this pull request as ready for review March 25, 2026 05:48
@khangronky khangronky requested a review from TaiVanNgo March 25, 2026 05:49
@github-actions

github-actions Bot commented Mar 25, 2026

Copy link
Copy Markdown

🔧 Biome Check Report

Total Issues Found: 0

🔧 Format Check Results

Metric Value Status
📁 Files Checked 1542 ✅ Complete
Errors 0 ✅ None
⚠️ Warnings 0 ✅ None
ℹ️ Info 0 ✅ None
📝 Total Issues 0 ✅ Passed

🔍 Lint Check Results

Metric Value Status
📁 Files Checked ✅ Complete
Errors 0 ✅ None
⚠️ Warnings 0 ✅ None
ℹ️ Info 0 ✅ None
📝 Total Issues 0 ✅ Passed

📋 Next Steps

🤖 Auto-generated by Biome Check workflow • Last updated: 6/11/2026, 9:46:22 AM

Copilot AI review requested due to automatic review settings April 2, 2026 11:00

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds an “Our Departments” feature to the marketing About page (with department detail cards and URL-driven selection) and surfaces a summarized department teaser section on the marketing homepage, alongside a broader pass replacing hard-coded brand hex colors with brand Tailwind tokens and improving semantic section markup.

Changes:

  • Add departments data model + new About “Departments” section with selectable department cards (and query-param selection).
  • Add a summarized departments section to the marketing landing page linking into About.
  • Replace various hard-coded brand hex colors with brand-* Tailwind tokens and adjust some sections from div to section.

Reviewed changes

Copilot reviewed 27 out of 42 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
apps/web/src/components/layouts/Footer.tsx Switch footer gradient color to brand token.
apps/web/src/app/[locale]/meet-together-menu-item.tsx Import ordering cleanup.
apps/web/src/app/[locale]/(marketing)/why-us.tsx Use semantic section + brand token colors.
apps/web/src/app/[locale]/(marketing)/summarized-departments.tsx New summarized department cards + links to About departments section.
apps/web/src/app/[locale]/(marketing)/projects/projects.tsx Replace brand hex colors with brand tokens.
apps/web/src/app/[locale]/(marketing)/projects/main-project.tsx Replace brand hex colors with brand tokens; minor markup tweaks.
apps/web/src/app/[locale]/(marketing)/projects/empty-state.tsx Replace brand hex colors with brand tokens.
apps/web/src/app/[locale]/(marketing)/page.tsx Add summarized departments section to marketing homepage.
apps/web/src/app/[locale]/(marketing)/neo-generator/hero.tsx Replace brand hex colors with brand tokens; class order tweaks.
apps/web/src/app/[locale]/(marketing)/neo-generator/client.tsx Import ordering + brand token color update.
apps/web/src/app/[locale]/(marketing)/meet-together/neo-meet-header.tsx Replace brand hex colors with brand tokens; class order tweaks.
apps/web/src/app/[locale]/(marketing)/hero-section.tsx Use semantic section wrapper.
apps/web/src/app/[locale]/(marketing)/events.tsx Use semantic section wrapper.
apps/web/src/app/[locale]/(marketing)/contributors/client.tsx Import ordering + brand token color updates; class order tweaks.
apps/web/src/app/[locale]/(marketing)/blogs/client.tsx Refactor blog fetching into useEffect; brand token updates.
apps/web/src/app/[locale]/(marketing)/blogs/[id]/client.tsx Brand token updates for badge/button.
apps/web/src/app/[locale]/(marketing)/achievements/page.tsx Import ordering + brand token updates; class order tweaks.
apps/web/src/app/[locale]/(marketing)/achievements/achievement-dialog.tsx Brand token update for badge.
apps/web/src/app/[locale]/(marketing)/about/timeline-card.tsx Tailwind v4 gradient utility update + typography tweaks.
apps/web/src/app/[locale]/(marketing)/about/page.tsx Add Departments section to About page and adjust layout container.
apps/web/src/app/[locale]/(marketing)/about/members.tsx Section id + brand token updates + minor naming/markup tweaks.
apps/web/src/app/[locale]/(marketing)/about/member-card.tsx Card styling refresh + Tailwind v4 gradient utility update.
apps/web/src/app/[locale]/(marketing)/about/history.tsx Callback memoization + semantic section wrapper + brand token updates.
apps/web/src/app/[locale]/(marketing)/about/departments.tsx New interactive Departments section with query-param selection.
apps/web/src/app/[locale]/(marketing)/about/department-card.tsx New detailed department card layout.
apps/web/src/app/[locale]/(marketing)/about/data.ts Add departments data model + department content data.
apps/web/src/app/[locale]/(games)/neo-pacman/page.tsx Brand token updates in hero heading.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread apps/web/src/app/[locale]/(marketing)/blogs/client.tsx
Comment thread apps/web/src/app/[locale]/(marketing)/summarized-departments.tsx
Comment thread apps/web/src/app/[locale]/(marketing)/summarized-departments.tsx
Comment thread apps/web/src/app/[locale]/(marketing)/about/departments.tsx
Comment thread apps/web/src/app/[locale]/(marketing)/about/departments.tsx
Comment thread apps/web/src/app/[locale]/(marketing)/about/page.tsx
Comment thread apps/web/src/app/[locale]/(marketing)/about/departments.tsx
@khangronky khangronky removed the WIP Work in progress - NOT READY TO BE MERGED label Apr 2, 2026
@khangronky khangronky changed the title Add Our Department section for About Us page feat: add Our Department section for About Us page Apr 23, 2026
@TaiVanNgo TaiVanNgo marked this pull request as draft April 23, 2026 17:14
@khangronky khangronky marked this pull request as ready for review May 1, 2026 12:46
@khangronky

Copy link
Copy Markdown
Collaborator Author

Please review and temporarily merge this @TaiVanNgo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants