Skip to content

feat(landing): add Feishu community link - #6745

Open
kokisanai wants to merge 3 commits into
mainfrom
feat/feishu-community-link
Open

feat(landing): add Feishu community link#6745
kokisanai wants to merge 3 commits into
mainfrom
feat/feishu-community-link

Conversation

@kokisanai

@kokisanai kokisanai commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add a Feishu entry immediately after Discord in the Community navigation
  • add a prominent community button to the top-right navigation
  • route Simplified and Traditional Chinese versions to Feishu (飞书社群 / 飛書社群)
  • route every other language version to Discord (Discord)
  • open community destinations in a new tab

Feishu tracking link

https://od.kokiai.net/community/website

Validation

  • pnpm --filter @open-design/landing-page typecheck
  • pnpm --filter @open-design/landing-page test
  • pnpm --filter @open-design/landing-page build:static
  • verified zh output renders the Feishu CTA and tracking link
  • verified English and Japanese output render the Discord CTA and invite link
  • verified the Feishu tracking link returns HTTP 302 to the group invite

@github-actions

github-actions Bot commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

🚀 Landing page preview

This PR is deployed to a Cloudflare Pages preview — not staging or production:

@joeylee12629-star
joeylee12629-star self-requested a review August 11, 2026 09:27
@lefarcen lefarcen added size/XS PR changes <20 lines risk/medium Medium risk: regular code changes type/feature New feature labels Aug 11, 2026

@lefarcen lefarcen left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hey @kokisanai, the Summary section already gives enough context for the change itself. Could you add a Surface area section and tick UI for the landing-page header nav update so the PR body reflects the user-facing change? That should make it easier for the pool reviewer to scope this one.

@lefarcen lefarcen added size/S PR changes 20-100 lines and removed size/XS PR changes <20 lines labels Aug 11, 2026

@xxiaoxiong xxiaoxiong left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM. Adds a Feishu community entry to the landing-page header — both as a dropdown item (alongside Discord) and as a top-right CTA, with locale-aware routing (zh / zh-tw → 飞书社群, others → Discord). Tracking via od.kokiai.net/community/website. CSS uses the existing coral pill pattern, has :focus-visible outline, and opens in a new tab via ext props. Self-contained to landing-page.

@xxiaoxiong xxiaoxiong left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM — clean community link addition.

Verified: FEISHU URL constant added next to DISCORD, locale-aware switch (usesFeishuCommunity = locale === 'zh' || locale === 'zh-tw') drives the right-side CTA platform, with proper strings for 飞书社群 / 蹈書社群 / Discord. The dropdown menu gets both Discord and Feishu entries side-by-side. CSS uses --coral shared variable, includes hover state and focus-visible outline for accessibility. CTA has flex-shrink: 0 so it stays visible on narrow viewports.

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

Labels

risk/medium Medium risk: regular code changes size/S PR changes 20-100 lines type/feature New feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants