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

30분마다 업데이트가 있는지 확인합니다. #28

Merged
merged 1 commit into from
Nov 12, 2014
Merged

30분마다 업데이트가 있는지 확인합니다. #28

merged 1 commit into from
Nov 12, 2014

Conversation

EdenYoon
Copy link
Contributor

#26 이슈를 해결합니다.

@@ -40,7 +40,8 @@ class AppDelegate: NSObject, NSApplicationDelegate {
let ga = AnalyticsHelper.sharedInstance()
ga.beginPeriodicReportingWithAccount("UA-42976442-2", name:"올ㅋ사전", version: BundleInfo.Version)

SUUpdater.sharedUpdater().checkForUpdatesInBackground()
self.checkForUpdatesInBackground()
NSTimer.scheduledTimerWithTimeInterval(30 * 60, target: self, selector: Selector("checkForUpdatesInBackground"), userInfo: nil, repeats: true)
Copy link
Owner

Choose a reason for hiding this comment

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

Swift 에서는 selector를 정의하는데 있어 명시적으로 Selector("checkForUpdatesInBackground")를 쓰는 대신 "checkForUpdatesInBackground" 처럼 selector 이름을 문자열로 적는 것을 권장하고 있습니다.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

아.. 설명했었는데, 깜빡했군요. 수정!

@EdenYoon
Copy link
Contributor Author

다시 커밋했습니다.

@EdenYoon
Copy link
Contributor Author

에러 수정했습니다.

devxoul added a commit that referenced this pull request Nov 12, 2014
30분마다 업데이트가 있는지 확인합니다.
@devxoul devxoul merged commit b460f76 into devxoul:master Nov 12, 2014
@devxoul
Copy link
Owner

devxoul commented Nov 12, 2014

감사합니다!

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

Successfully merging this pull request may close these issues.

2 participants