Skip to content

Comments

Claude/fix blog redirect issue v qf xk#3755

Open
tyr542 wants to merge 165 commits intotangly1024:mainfrom
tyr542:claude/fix-blog-redirect-issue-vQfXK
Open

Claude/fix blog redirect issue v qf xk#3755
tyr542 wants to merge 165 commits intotangly1024:mainfrom
tyr542:claude/fix-blog-redirect-issue-vQfXK

Conversation

@tyr542
Copy link

@tyr542 tyr542 commented Feb 8, 2026

尽量按此模板PR内容,或粘贴相关的ISSUE链接。

已知问题

  1. (示例)版本号管理不规范
    • 版本号直接写在环境变量中,容易出错
    • 多处维护版本号,可能不一致

解决方案

  1. (示例)将版本号管理从 .env.local 迁移到 package.json
    • 统一从 package.json 读取版本号
    • 使用 IIFE 优雅处理版本号获取逻辑
    • 保持向后兼容,支持环境变量覆盖

改动收益

  1. (示例)更规范的版本管理
    • 统一从 package.json 读取
    • 保持与 npm 生态一致
    • 减少人为错误

具体改动

  1. (示例)blog.config.js
    • 移除原有的静态版本号配置
    • 在文件末尾添加动态版本号获取逻辑
    • 保持向后兼容,优先使用环境变量
    • 添加错误处理和默认值

测试确认

  • 本地开发环境测试通过
  • 生产环境构建测试通过
  • 版本号正确显示
  • 环境变量配置正常工作

tyr542 and others added 29 commits January 12, 2026 20:45
問題:過度移除 Tailwind class 導致排版跑掉
- group class 是 Tailwind group-hover 功能必要的
- 佈局相關的 flex, w-full, md:w-7/12 等不應移除

修正策略:
- 保留所有佈局相關的 Tailwind class
- 只抽取「顏色」相關的樣式到 custom.css(如 accent-hover, tag-pill)
- 不再過度抽取佈局樣式
這三個文件不應該被修改,只有顏色相關的樣式才應該整合到 custom.css
- Quote:移除背景和引號水印,改用透明背景 + 強調色左邊框 + 斜體
- Callout:移除紅色標題和白灰漸層,改用淡米灰背景 + 強調色左邊框 + 深墨色宋體標題
- 兩者視覺上有明確區隔:Quote 更輕盈(透明),Callout 有淡背景
- Quote:加淡米褐色背景 + 左邊框,更有存在感
- Callout:改用細邊框包圍,移除強制背景色讓 Notion 原生顏色生效
- 新增 .text-accent, .hover-accent, .bg-accent 等強調色 utility
- 新增 .nav-blur 導航模糊效果
- 新增 .pagination-item 分頁按鈕樣式
- 新增 .header-hero z-index 設定
- 新增 .busuanzi-hidden 初始隱藏狀態
- 移除重複的選擇器定義
- hover:text-[#8c7b75] → accent-hover
- tag 樣式改用 tag-pill class
Notion 於 2026/2/7 更新了內部 API 回應格式,資料多了額外的巢狀層級,
導致 NotionNext 無法正確解析文章資料,所有文章頁面顯示 404。

修復內容(參考上游 tangly1024/NotionNext commit 3219205):
- 新增 normalizeNotionMetadata: 處理 metadata 多層嵌套
- 新增 normalizeCollection: 解包 collection 最多 3 層嵌套
- 新增 normalizeSchema: 標準化 schema 欄位定義
- 新增 normalizePageBlock: 解包 page block 最多 5 層嵌套
- 新增 cleanBlocksWithWarn: 清理 blockMap 中無效的 block

https://claude.ai/code/session_01UKxFp8MwJcNcMG5yvoQ4np
@vercel
Copy link

vercel bot commented Feb 8, 2026

@claude is attempting to deploy a commit to the tangly1024's projects Team on Vercel.

A member of the Team first needs to authorize it.

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