-
Notifications
You must be signed in to change notification settings - Fork 9
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
Bi weekly webhook #12
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for your quick implementation!
I have left some comments below.
BTW, can we modify the message created by this Discord bot? If that is possible, we will be able to update the CodePair link after the bot generates the message.
It seems difficult to modify messages created by the Webhook. For now, I think we can add the CodePair document URL using a Discord thread. In the future, I’d like to implement a document creation REST API in CodePair and automate the bi-weekly document creation process. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
It seems difficult to modify messages created by the Webhook. For now, I think we can add the CodePair document URL using a Discord thread.
Then how about just remove the link for CodePair link for now, and mention it when we actually start the weekly sync?
We have bi-weekly meetings on Saturdays.
Since it’s a hassle to manually send reminders every time, I added a GitHub Action to automatically send the message every other week.
Before merging, make sure to add
DISCORD_WEBHOOK
to the repository secrets.