Skip to content
This repository has been archived by the owner on Nov 17, 2022. It is now read-only.

feat: support static runtime #609

Merged
merged 5 commits into from
Oct 24, 2022
Merged

feat: support static runtime #609

merged 5 commits into from
Oct 24, 2022

Conversation

ClarkXia
Copy link
Collaborator

@ClarkXia ClarkXia commented Oct 20, 2022

statc runtime 说明:
对于一些支持在 src/app 入口就可以使用的 API,如 request 的运行时能力,需要在应用数据 / 路由数据加载获取前就需要加载,否则在 getAppData 中将无法享受到统一的配置能力

Close #604 #605

@vercel
Copy link

vercel bot commented Oct 20, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
ice-v3 ✅ Ready (Inspect) Visit Preview Oct 24, 2022 at 2:46AM (UTC)

chenjun1011
chenjun1011 previously approved these changes Oct 21, 2022
await Promise.all(runtimeModules.map(m => runtime.loadModule(m)).filter(Boolean));

const { Document } = renderOptions;
const appContext = runtime.getAppContext();
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

renderDocument 里的 APPContext 要统一下吗?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

应该不需要,之前看了下 Document 渲染应该不依赖运行时能力,理论上 document 的渲染逻辑要再精简下

@ClarkXia ClarkXia merged commit c098149 into release-next Oct 24, 2022
@ClarkXia ClarkXia deleted the fix/runtime branch October 24, 2022 07:17
@ClarkXia ClarkXia mentioned this pull request Oct 27, 2022
16 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants