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

sync page support logId #70

Closed
dead-horse opened this issue Dec 11, 2013 · 5 comments
Closed

sync page support logId #70

dead-horse opened this issue Dec 11, 2013 · 5 comments
Assignees
Milestone

Comments

@dead-horse
Copy link
Member

@dead-horse dead-horse commented Dec 11, 2013

GET http://cnpmjs,org/sync/connect 会发起一次同步。
同步开始之后,url变成http://cnpmjs,org/sync/connect#logid=100

GET http://cnpmjs,org/sync/connect#logid=100将不会重新发起一次同步,将会直接开始获取logid=100的同步日志

这样sync由于请求npmjs.org/connect超时导致未能同步的时候,直接刷新页面就可以重试,如果开始同步,刷新也不会重新触发多一次同步

@ghost ghost assigned dead-horse Dec 11, 2013
@dead-horse
Copy link
Member Author

@dead-horse dead-horse commented Dec 11, 2013

@fengmk2
Copy link
Member

@fengmk2 fengmk2 commented Dec 11, 2013

最好是 POST http://cnpmjs,org/sync/connect 会发起一次同步

GET http://cnpmjs,org/sync/connect 没有logid就跳到package 页面

@dead-horse
Copy link
Member Author

@dead-horse dead-horse commented Dec 11, 2013

pic
现在点其他页面的Sync我是直接跳转到/sync/connect,由这个页面发起PUT /sync/connect的请求开始同步的,这样会不会方便点?

@fengmk2
Copy link
Member

@fengmk2 fengmk2 commented Dec 11, 2013

那就先这样做吧

@dead-horse
Copy link
Member Author

@dead-horse dead-horse commented Dec 11, 2013

这样如果想要同步一个模块,也可以直接打开cnpmjs.org/sync/connect这个页面就可以了

dead-horse added a commit that referenced this issue Dec 11, 2013
fengmk2 added a commit that referenced this issue Dec 11, 2013
refine sync page, fiexd #70
@fengmk2 fengmk2 closed this Dec 11, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.