Skip to content

feat(posts): add EP56 急急急 useLinkStatus podcast episode#53

Merged
AnnatarHe merged 1 commit intomasterfrom
feat/add-ep56-useLinkStatus
Mar 8, 2026
Merged

feat(posts): add EP56 急急急 useLinkStatus podcast episode#53
AnnatarHe merged 1 commit intomasterfrom
feat/add-ep56-useLinkStatus

Conversation

@AnnatarHe
Copy link
Copy Markdown
Member

Summary

  • Add new podcast episode EP56 about Next.js useLinkStatus hook
  • Includes 小宇宙, YouTube, and Bilibili links
  • Shownotes with reference to Next.js docs

Test plan

  • pnpm astro check passes with 0 errors
  • Preview with pnpm dev and verify episode renders correctly

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@AnnatarHe AnnatarHe temporarily deployed to feat/add-ep56-useLinkStatus - asynctalk-website PR #53 March 8, 2026 16:08 — with Render Destroyed
@AnnatarHe AnnatarHe merged commit 88eca24 into master Mar 8, 2026
1 check passed
@AnnatarHe AnnatarHe deleted the feat/add-ep56-useLinkStatus branch March 8, 2026 16:08
@claude
Copy link
Copy Markdown

claude Bot commented Mar 8, 2026

EP56 PR Review

整体来看,这个 PR 遵循了现有的内容结构和命名规范,可以合并。

符合规范: 文件命名遵循 epXX.mdx 模式;所有 schema 必填字段均已填写(title, author, publicationDate, categories, status);guid、episodeNumber、slug 均与 EP56 对应正确;season: 3 与近期 episodes 一致。

问题一:size: 0 和 duration: 0 是占位值。 这两个字段对 RSS feed 元数据有影响,订阅客户端(如 Apple Podcasts、小宇宙)可能依赖它们展示时长或下载进度。虽然 ep54/ep55 也是如此,建议补充真实值。

问题二:biliUrl 使用协议相对 URL(//)。 ep54 使用 https://,ep55 和 ep56 使用 //。建议统一使用 https:// 前缀,与 ep54 保持一致,避免潜在的混合内容问题。

问题三:url 与 xyzLink 重复。 两个字段指向相同 URL。url 字段不在 content schema 中,这与现有 episodes 一致,无需修改,但可以在项目文档中说明各字段用途。

次要建议: Test plan 中 pnpm dev 预览一项未打勾,建议合并前确认本地渲染正常(YouTube/Bilibili 嵌入)。Shownotes 内容较简短,如有更多背景可提升内容价值。

总结: 核心内容没有问题,建议修复 biliUrl 的协议前缀(// 改为 https://)后合并。

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.

1 participant