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: react to message #10

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ogadra
Copy link

@ogadra ogadra commented Aug 26, 2022

close #9
image

Assigneessが割り当てられたことをトリガーに、Discordで該当IssueURLが含まれたコメントに対してリアクションを行う。

現状は以上の理由で動作しない。

  • 環境変数(DISCORD_BOT_TOKEN及びDISCORD_READY_ISSUE_CHANNEL)がJSから参照できていない。
  • Botにチャンネルの投稿閲覧権限が与えられていない。

@ogadra ogadra self-assigned this Aug 26, 2022
@ogadra ogadra requested a review from exKAZUu August 26, 2022 08:41
@ogadra
Copy link
Author

ogadra commented Aug 26, 2022

差分が大量に出ていますが、これはGitHub Actionsで動作させるためにnode_moduleをすべて含んでしまっているからです。
(もしかしたら含めなくても動かす方法があるかもしれませんが)
該当ファイルは.github/actions/assignees-notify/src/index.tsなので、それを中心に見ていただければ…と思います。

@exKAZUu
Copy link
Member

exKAZUu commented Sep 7, 2022

TODO: WillBooster/reusable-workflows に移しつつ、 node_modules を含めずに済むように直す。

@exKAZUu exKAZUu self-assigned this Sep 8, 2022
@exKAZUu
Copy link
Member

exKAZUu commented Sep 18, 2022

@ogadra dist/index.js のもととなったコードも入れてもらえないだろうか〜

@ogadra
Copy link
Author

ogadra commented Sep 21, 2022

@exKAZUu src/index.tsですが、githubが重くて表示されてないみたいですね…
fetchしてローカルで見れば見られるはずです

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.

feat: reaction to ready issue when an assignee appears
2 participants