Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Notion config的配置项可以使用对象数组等复杂结构和HEO主题的站点信息文案支持从Notion配置 #3077

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

qixing-jk
Copy link
Contributor

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

已知问题

  1. Notion config不支持复杂结构的对象配置的,导致部分文案只能写死
  2. HEO主题站点信息部分文案不支持从Notion配置

解决方案

  1. 改为使用JSON parse解析而非eval

改动收益

  1. 更多的配置项可从Notion获取

具体改动

  1. 详见commits

测试确认

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

Copy link

vercel bot commented Dec 21, 2024

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

A member of the Team first needs to authorize it.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant