Skip to content
This repository has been archived by the owner on Jun 2, 2024. It is now read-only.

Add missing properies and sync missing star users. fixed #235 #244

Merged
merged 1 commit into from
Mar 4, 2014

Conversation

fengmk2
Copy link
Member

@fengmk2 fengmk2 commented Mar 4, 2014

  • add readmeFilename and missing created, modified time.
  • Sync star users on pkg sync flow
  • Show star users on registry.show
  • Show star users on web package show
  • Need to create new table module_star

@fengmk2 fengmk2 added this to the v0.3 koa milestone Mar 4, 2014
* add readmeFilename and missing created, modified time.
* Sync star users on pkg sync flow
* Show star users on registry.show
* Show star users on web package show
* Need to create new table module_star
}

that.log(' [%s] saving %d star users', name, missingStarUsers.length);
missingStarUsers.forEach(function (username) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

express有200多个starUser.. 第一次同步的时候数据库没问题?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

没问题的,没压力,对于mysql。。。

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

好 合并了, 今晚我改sync worker

dead-horse added a commit that referenced this pull request Mar 4, 2014
Add missing properies and sync missing star users. fixed #235
@dead-horse dead-horse merged commit 4460810 into master Mar 4, 2014
@dead-horse dead-horse deleted the missing-propertys branch March 4, 2014 13:35
@fengmk2
Copy link
Member Author

fengmk2 commented Mar 4, 2014

LGTM哈哈

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

Successfully merging this pull request may close these issues.

2 participants