angular-bbs / user-ui

Angular BBS的前端,基于Angular 2 + bootstrap 4
MIT License
47 stars 27 forks source link

feature proposal: 嫁接universal-starter #43

Closed rxjs-space closed 7 years ago

rxjs-space commented 8 years ago

这个PR仅仅是proposal,太多地方需要完善,甚至需要重做。 嫁接universal-starter,主要目的是让搜索引擎能看到这个网站的具体内容。

PR主要内容:

  1. 引入universal-starter,但不影响之前的开发流程,即npm start还是和原来一样
  2. 首页跳转至library/article,立即载入LibraryModule

使用方法:

已知问题:

  1. AppModule(用来开发), AppNodeModule), AppBrowserModule三者有太多重复,更改其中一个文件,可能要同时更改另外两个。考虑重置app结构,引入AppCommonModule, 类似webpack.common.js;
  2. 不知道如何在server side使用webpack载入scss,现在用的是raw-loader,效果就是这个scss不起作用,载入时看不到style;
  3. 如果网站用户在浏览器输入library/article/title,服务器端报错, Cannot read property 'unsubscribe' of null,服务器render失败,转到客户端render,且不影响后续运行;
  4. 如果网站用户在浏览器输入about,会有闪烁,因为AboutModule是lazy loading,需要考虑重置app结构。
rxjs-space commented 7 years ago

let's see what angular 4 will offer first.