Skip to content

content: 华南区智能基座社团交流活动文章及首页动态热图修复 - #13

Merged
ZreXoc merged 6 commits into
hitszosa:mainfrom
ZreXoc:content/huawei-songshan-articles
Sep 18, 2026
Merged

ZreXoc merged 6 commits into
hitszosa:mainfrom
ZreXoc:content/huawei-songshan-articles

Conversation

@ZreXoc

@ZreXoc ZreXoc commented Sep 17, 2026

Copy link
Copy Markdown
Member

变更内容

新文章:华南区智能基座社团交流活动(2026-09-12)

  • 新增 content/articles/2026/09/huawei_songshan/:活动通告 index.mdx + 后记 afterword.mdx(作者 XiC)
  • 11 张配图已统一压缩(长边 ≤2000px、JPEG q82、去 EXIF),总大小约 3.4MB(原 55MB)
  • 后记通过 RelatedLink 与正文互链

修复:首页 UPDATE MAP 展示范围

  • 热图 126 天窗口原来从最老一条动态所在周起算,最新动态可能落在窗口外
  • 现改为锚定最新动态所在周的周日为结束日,向前倒推 126 天;无动态时回退到当前时间
  • 效果:窗口始终覆盖最近的动态(当前为 2026-05-10 ~ 2026-09-13)

Copilot AI lite review requested due to automatic review settings September 17, 2026 15:25

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.

🟡 Changes recommended

Unresolved moderate findings affect article links, image paths, and homepage date-window boundaries.

Get a fresh assessment by requesting another Copilot review.

Pull request overview

Adds a South China Intelligent Base community exchange article and afterword, plus an UPDATE MAP window adjustment.

Changes:

  • Adds the article, afterword, and 11 compressed images.
  • Adds reciprocal article links.
  • Revises homepage heatmap date-window anchoring.
File summaries
File Changes and final findings
content/articles/2026/09/huawei_songshan/index.mdx Adds the activity article. Moderate findings: fix bare asset paths, remove the duplicate H1, and add the afterword trailing slash. Nit: preserve XiC capitalization.
content/articles/2026/09/huawei_songshan/afterword.mdx Adds the afterword. Moderate findings: correct both article links to parent-relative paths and prefix all local image paths with ./assets/. Nit: correct the noted Chinese wording.
apps/landing/src/pages/index.astro Adjusts heatmap anchoring. Moderate findings: exclude future-dated events from the anchor and use the exclusive next-Sunday boundary for the 126-day window.
Review details

Suppressed comments (8)

apps/landing/src/pages/index.astro:46

  • The new article makes the latest record 2026-09-12 (Saturday), so this expression sets heatmapEnd to 2026-09-12, not the Sunday boundary claimed in the PR description; the generated cells therefore end on 09-12 instead of the stated 09-13 boundary. Treat the end as the exclusive next Sunday and subtract 126 days when computing the start so the 126 cells remain aligned to full weeks.
    latestRecord.getUTCDate() + (6 - latestRecord.getUTCDay()),

content/articles/2026/09/huawei_songshan/afterword.mdx:45

  • The three content-local image URLs omit the leading ./; unlike the existing article convention, they will be emitted as browser-relative URLs rather than resolved from this article's assets directory. Prefix each with ./assets/ so the images are included in the build.
![风景0](assets/风景0.jpg)
![风景1](assets/风景1.jpg)
![风景2](assets/风景2.jpg)

content/articles/2026/09/huawei_songshan/afterword.mdx:58

  • These content-local image URLs have the same missing ./ prefix and will not resolve to the article assets. Prefix both paths with ./assets/.
![饭1](assets/饭1.jpg)
![饭2](assets/饭2.jpg)

content/articles/2026/09/huawei_songshan/afterword.mdx:72

  • This content-local image URL is missing the ./ prefix used by the existing article convention, so the image will not be resolved from the article's assets directory.
![讨论结果](assets/讨论结果.jpg)

content/articles/2026/09/huawei_songshan/afterword.mdx:41

  • “园区据欧式风格打造” is not grammatical Chinese; the intended wording appears to be “园区以欧式风格打造”.
华为溪村坐落于东莞,园区据欧式风格打造,风景秀丽环境优美。

content/articles/2026/09/huawei_songshan/index.mdx:24

  • This content-local image path is missing the leading ./ used by the existing article image references (for example, content/articles/2026/08/OSA_in_OSS_Forum/index.mdx:28). Without it, Astro treats the URL as a browser-relative path instead of resolving/bundling the image from this article's assets directory, so the image will be broken. Prefix this path with ./ and apply the same correction to the other bare assets/ references in this file.
![xc的自我介绍](assets/xc的自我介绍.jpg)

content/articles/2026/09/huawei_songshan/index.mdx:10

  • The article template already renders the frontmatter title as the page's <h1> (apps/landing/src/pages/articles/[...id].astro:56-60), so this body-level # adds a second top-level heading with a different title. Remove it and keep the existing ## headings for the article sections.
# 哈工深华为“智能基座”社团参加 2026 年华南高校“智能基座”学生社团运营研讨会

content/articles/2026/09/huawei_songshan/index.mdx:41

  • This target omits the trailing slash, while the app emits slash-terminated article URLs and the static route is a directory. Make the link ./afterword/ so the reciprocal relative links resolve consistently instead of depending on host-specific trailing-slash redirects.
  href="./afterword"
  • Files reviewed: 3/14 changed files
  • Comments generated: 4
  • Review effort level: Lite

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

Comment thread apps/landing/src/pages/index.astro Outdated
Comment on lines +39 to +41
const latestRecord = updateRecords.length
? new Date(Math.max(...updateRecords.map((entry) => entry.date.getTime())))
: now;
---

<RelatedLink
href="./"
> 9 月 12 日,哈尔滨工业大学(深圳)华为“智能基座”社团社长 _XiC_ 与 _StrayRN_ 二人赴华为溪村,参加 2026 年华南高校“智能基座”学生社团运营研讨会。来自华南地区 9 所入选教育部—华为“智能基座”试点建设清单高校的师生代表齐聚一堂,围绕社团建设与开源生态展开深入交流。


这是有关于 [华南区智能基座社团交流活动](../huawei_songshan/) 一文的后记。

<RelatedLink
href="./afterword"
description="后记 by Xic"
@ZreXoc
ZreXoc merged commit 4e4419e into hitszosa:main Sep 18, 2026
2 checks passed
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