Skip to content

Fix/notion api compat 2026 06#4214

Open
daheitian wants to merge 38 commits into
notionnext-org:mainfrom
daheitian:fix/notion-api-compat-2026-06
Open

Fix/notion api compat 2026 06#4214
daheitian wants to merge 38 commits into
notionnext-org:mainfrom
daheitian:fix/notion-api-compat-2026-06

Conversation

@daheitian

Copy link
Copy Markdown

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

已知问题

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

解决方案

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

改动收益

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

具体改动

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

测试确认

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

用户文档(docs/user-guide/

若本 PR 修改 docs/user-guide/docs/developer/ 中与站长相关的说明,可勾选「不适用」并跳过本节。

  • 不适用(无文档改动)
  • 已按 维护工作流 自检
  • 路径符合 docs/user-guide/ 目录约定
  • 已更新 user-guide/README.md(新增/移动文章时)
  • 已更新 ARTICLE_INDEX.md(新 slug 或路径变更时)
  • 环境变量名与 conf/*.config.js 一致(若文档涉及配置)
  • 示例中无真实 Token、.env、私有 ID
  • 保留或更新了「原文链接」(若源自 docs.tangly1024.com)

文档说明(可选):对应官方 slug / URL、是否与功能 PR 配套

@daheitian daheitian requested a review from tangly1024 as a code owner June 26, 2026 01:47
@vercel

vercel Bot commented Jun 26, 2026

Copy link
Copy Markdown

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

A member of the Team first needs to authorize it.

@netlify

netlify Bot commented Jun 26, 2026

Copy link
Copy Markdown

Deploy Preview for notionnext-netlify failed.

Name Link
🔨 Latest commit a616047
🔍 Latest deploy log https://app.netlify.com/projects/notionnext-netlify/deploys/6a3dda47d72c40000880a188
@githubdudu

Copy link
Copy Markdown
Contributor

Apparently, this PR is mean to merge with author's own repo, not the NotionNext-org one.
Besides the PR contains:

  1. Personal .env.local
  2. Personal blog.config.js

3 And This branch is 38 commits ahead of and 1244 commits behind notionnext-org/NotionNext:main.

It can be closed.

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

Labels

None yet

3 participants