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

v2 #5

Merged
merged 8 commits into from
Nov 10, 2015
Merged

v2 #5

merged 8 commits into from
Nov 10, 2015

Conversation

rockdai
Copy link
Owner

@rockdai rockdai commented Nov 10, 2015

  • use Douban API instead of fetch page.
  • make crawler simple

@alsotang
Copy link
Collaborator

巴老板玩上 ES6 了啊。

跑得动就好啦。反正都是只读的。

部署的事情..巴老板换个地址吧

@rockdai
Copy link
Owner Author

rockdai commented Nov 10, 2015

@alsotang 为毛要换地址?你不是加我权限了嚒,我自己可以部署啊。

@alsotang
Copy link
Collaborator

噢噢。。原来你可以部署啊。。好久没用 heroku 了。。

alsotang added a commit that referenced this pull request Nov 10, 2015
@alsotang alsotang merged commit 004da91 into master Nov 10, 2015
@rockdai rockdai deleted the v2 branch November 10, 2015 15:14
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

Successfully merging this pull request may close these issues.

2 participants