Skip to content

chore(website): hero section and i18n content update - #21

Merged
JusterZhu merged 1 commit into
mainfrom
chore/official-website-content
Aug 29, 2026
Merged

chore(website): hero section and i18n content update#21
JusterZhu merged 1 commit into
mainfrom
chore/official-website-content

Conversation

@JusterZhu

Copy link
Copy Markdown
Contributor

预先存在的 official_website 内容改动(hero + i18n),独立提交避免混入功能分支。

Copilot AI lite review requested due to automatic review settings August 29, 2026 04:53
@JusterZhu
JusterZhu merged commit e5dd3bd into main Aug 29, 2026
2 checks passed

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 updates the official website hero section to embed a Bilibili demo video and introduces new i18n strings needed to label the video link in both English and Chinese.

Changes:

  • Add video.watchOnBilibili i18n entries for en and zh.
  • Replace the hero image with a Bilibili <iframe> embed and add a “Watch on Bilibili” external link under it.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
official_website/src/i18n.ts Adds new video.watchOnBilibili strings for EN/ZH dictionaries.
official_website/src/components/hero.astro Replaces hero image with a Bilibili video embed and adds a localized link below the video.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment on lines 4 to +5
const dict = getDict(Astro.currentLocale);
const videoTitle = "是的,开源了一个NAS系统fortOS";
Comment on lines +45 to +49
scrolling="no"
border="0"
frameborder="no"
framespacing="0"
allowfullscreen="true"
Comment on lines +54 to +57
<a
href="https://www.bilibili.com/video/BV1uNuR6fETY"
target="_blank"
rel="noopener"
@JusterZhu
JusterZhu deleted the chore/official-website-content branch August 29, 2026 04:56
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.

2 participants