Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

几点建议 #10

Closed
qfdk opened this issue Apr 7, 2017 · 9 comments
Closed

几点建议 #10

qfdk opened this issue Apr 7, 2017 · 9 comments

Comments

@qfdk
Copy link
Contributor

qfdk commented Apr 7, 2017

Hey

首先感谢作者的努力, 其次感谢大家对此项目的支持 :)

一下有几点意见,不知道意下如何。

  1. Dockerfile 能不能判断一下所在区域 ?上来就是cnpm让在vps跑的人很尴尬

  2. 搜索 我想加入Elasticsearch 因为看到了有Mongodb 可以同步数据进Elasticsearch

mongo-connector -m 127.0.0.1:27017 -t 127.0.0.1:9200 -d elastic_doc_manager
  1. 图床也可以照顾一下海外的盆友,qiniu 一直让填个手机号并且速度不是很快。我觉得imgur是个不错的选择 :)

  2. 前端这里可以考虑加个文件夹 比如默认用Default 然后用户可以自己设计一下主题。

翻了一下作者close的issues 回答都很详细,再次感谢。

有空了 会给这个项目出一份力 :)

@smallpath
Copy link
Owner

谢谢关注🤓

  1. Dockerfile这个问题确实存在,cnpm之后准备换成yarn。
  2. es我一直在犹豫是否上,因为部署太重量级了,感觉之后可以弄个可选的部署开关
  3. 这个了解了,海外图床会顺便和i18n一起做
  4. 前端算是直接拿RESTful api自己做的,我也建议有兴趣的用户直接拿api新建一个项目,而不是被局限在原主题的模板里。

@qfdk
Copy link
Contributor Author

qfdk commented Apr 7, 2017

@smallpath 刚测试了一下,打包之后本地跑了一下 出现了这样的问题,导致无法切换页面
qq 20170407125252
觉得还是有玩儿头的 :)
另外可以考虑上一下travis,,这样贡献代码风格有个统一
https://standardjs.com/

es部署那边直接做个docker 镜像就好了 :) 弄个开关比较靠谱,只要填上es的url 就用es 如果空就用谷歌。这个可以当做一个新特性,喜欢折腾的同学可以玩儿这个。

PS: 有没有个直接可以讨论的地方,比如建立个QQ群或者telegram的群?

@smallpath
Copy link
Owner

smallpath commented Apr 7, 2017

看起来像是没开代理的样子,npm run build的模式下需要配置nginx代理,开发情况下用npm run dev就可以了,这时会webpack自动代理。

CI是一定会上的,最近三个月我都有时间,会一直迭代本项目

@qfdk
Copy link
Contributor Author

qfdk commented Apr 7, 2017

@smallpath admin我是npm run dev front是 直接npm start 就是没开代理也不应该出现这样的错误,好奇脸

开个能讨论的地方吧:)

@smallpath
Copy link
Owner

没上服务端渲染的SPA只能靠nginx代理,不过你的话倒是提醒我了,由于front默认主题是SSR的,确实可以直接node代理过去不走nginx。我尝试下

@smallpath
Copy link
Owner

smallpath commented Apr 7, 2017

Hi,front的代理已经做好了,你可以拉一下最新代码,现在生产环境下已经可以不需要nginx了。拉完别忘了npm install重新装一遍依赖后,再复制一下最新的配置文件。

之前测试生产环境时需要自己配hosts,并且自己配的hosts还没法启用service worker,现在就舒服多了

@qfdk
Copy link
Contributor Author

qfdk commented Apr 7, 2017

@smallpath ok 谢谢~ 我重新fork一份 :)

@qfdk
Copy link
Contributor Author

qfdk commented Apr 10, 2017

@smallpath 图床想了一下 有个叫做sm.ms 的似乎还是不错的,可以考虑接入一下,那个也没有什么apikey的样子,不用配置了。

@smallpath
Copy link
Owner

HI,公共图床可以加上,搜索服务我准备在后端重构后加上,计划先支持es和你提的algolia,那个PR和这个issue我先关了,有其他的话随时开这个issue,或者在我刚刚开的todo列表这个issue回复就好

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

No branches or pull requests

2 participants