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

[FEAT] 24시마다 스케줄잡 추가 #87

Merged
merged 14 commits into from
Jan 17, 2022
Merged

Conversation

xxeol2
Copy link
Contributor

@xxeol2 xxeol2 commented Jan 17, 2022

✅ Default Checklist

  • check branch

  • set Labels

  • set Reviewers


📕 Task

  • 24시마다 스케줄잡 추가

😳 Issue

  • Help me..

@xxeol2 xxeol2 self-assigned this Jan 17, 2022
Copy link
Member

@youngkwon02 youngkwon02 left a comment

Choose a reason for hiding this comment

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

멋있다 설희~~ Join 연산 써서 쿼리도 깔끔하게 몇개 없구,,!!
우리팀 왤케 멋지지~?!? 몇일 안남았으니까 남은 시간도 홧팅!!!

const schedule = require('node-schedule');
const funcs = require('./funcs.js');

const job = schedule.scheduleJob('* 0 * * *', function() {
Copy link
Member

Choose a reason for hiding this comment

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

@xxeol2
설희 혹시 Cron 표현법이 초, 분, 시, 일, 월, 요일 해서 총 6자리로 알고있는데, * * 0 * * * 로 써야하는거 아냥??!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

초가 Optional이라 필요가 없는느낌인가봥!
그리고 timezone 반영해서 * 15 * * * 로 수정해뒀슴다!

Copy link
Member

Choose a reason for hiding this comment

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

앗 그렇구낭!!
오키~~ 고마워~~~

@xxeol2 xxeol2 merged commit 46eecfe into develop Jan 17, 2022
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.

None yet

2 participants