Skip to content

2度目のhook通知を17時30分にセットした#44

Draft
uesaka-seiya wants to merge 3 commits into
masterfrom
second-notice
Draft

2度目のhook通知を17時30分にセットした#44
uesaka-seiya wants to merge 3 commits into
masterfrom
second-notice

Conversation

@uesaka-seiya

@uesaka-seiya uesaka-seiya commented Nov 16, 2020

Copy link
Copy Markdown
Collaborator

close #43

試しに↓のように書いてみたところ30秒毎に動くことは確認できました。

    @Scheduled(cron = "0 * * * * 1-5", zone = "Asia/Tokyo")
    @Scheduled(cron = "30 * * * * 1-5", zone = "Asia/Tokyo")
    public void postToHook() {
        // 処理
    }

もっとエレガントなやり方とかあれば教えてください。

@uesaka-seiya uesaka-seiya self-assigned this Nov 16, 2020
RyuyaMaeda
RyuyaMaeda previously approved these changes Nov 19, 2020

@RyuyaMaeda RyuyaMaeda left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@uesaka-seiya
レビューしました、良いと思います

@RyuyaMaeda RyuyaMaeda dismissed their stale review November 19, 2020 23:16

一点確認したいことがあったので一旦取り消します:bow:

@RyuyaMaeda

Copy link
Copy Markdown
Collaborator

二度目のhook通知で次の掃除当番に通知が行く気がします。
微妙ですが、10時に実行されるgetCleaner()は別で考えた方がいいかもしれないです。
こちらで他の掃除忘れ防止策についても考えてみたいと思います。

@uesaka-seiya

Copy link
Copy Markdown
Collaborator Author

そうでした(その日の夕方に掃除当番交代してもらっちゃ困る) 🙈
getCleaner()実行のタイミングを考えてみます

@uesaka-seiya uesaka-seiya marked this pull request as draft November 30, 2020 09:05
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.

夕方にもHOOK通知をする

2 participants