定义 initialize 阶段；解决平台与开源版 API 分叉问题 #62
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…t as below: Centralize most initialization works, also some adjustment done to webpack configs and request.js, finish merging platform-api branch :fire: remove ace editor related codes Fix process.env Fix webpack config Fix scrollbar style Fix web socket Add a switch version button :hammer: introcude initialize lifecycle stage to app
|
补充：
这个 pr 同时包含了杨臻做的 platform-api 分支，因为要抹掉开发参数历史的关系，相关 commit 被 squash 了. 内容包含：
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
定义 initialize 阶段，把主要的初始化工作都移入 intialize.js 文件集中进行。现在利用
steputil function 强制这个初始化过程是串行的，这样比较符合初始化的业务需求，也方便之后做初始化进度条。整理了平台版与开源版由于 API 差异造成的逻辑分叉，统一用 if (config.isPlatform) 来标记这些差异，下一步或由后端改 API 来尽量磨平这些差异。同时把原先开源版上 workspaces list 的页面分出来自己成一个 entry point，以求跟平台版统一。