-
Notifications
You must be signed in to change notification settings - Fork 2
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
WebhookをBotに置き換え #553
base: main
Are you sure you want to change the base?
WebhookをBotに置き換え #553
Conversation
メモ #545 でもtraQのbotを作ってaccess tokenを設定していてコンフリクトしているので何とかする |
type Post struct { | ||
MessageID uuid.UUID | ||
EventID uuid.UUID | ||
} |
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.
eventにtraQのmessageを紐づけるなら新しくテーブルを作らなくてもeventsテーブルにカラムを追加するのでもいいかもしれないです
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.
少しコメントしました.
database のスキーマは確かに新しいカラムを増やすだけでも良いのかなと私も思いました
コメント失敗した
#545 がマージされてTRAQ_ACCESS_TOKENが使われるようになったのでTRAQ_BOT_TOKENではなくこれを使ってほしいです:pray: |
resolves #377
イベントの作成時、メッセージを投稿しそのmessageIDとeventIDがpostsテーブルに入ります
メッセージにスタンプが付けられた際、postsテーブルを参照して対応するイベントの出席状態が変化します
migrationはよくわかってないので雰囲気で書いてます
自動生成に失敗しているので今のところconverterは手実装になっています:ayase_iyaa: