jaoafa / jaotan.ts

👧 jao Gamers Club Official Discord Bot for TypeScript
https://jaoafa.github.io/jaotan.ts/
MIT License
1 stars 4 forks source link

meeting_voteの不具合 #136

Closed book000 closed 1 year ago

book000 commented 1 year ago
book000 commented 1 year ago

@yuuahp すみません、もし出来たら以下のコード確認いただいて、問題ありそうなところ指摘いただきたいです。難しかったら言ってください。

過半数処理がおかしいのかと推測していますが、実際のところちょっとよくわからん状態です 不具合の詳細は新しいmeetingチャンネル参照ください

https://github.com/jaoafa/jaotan.ts/blob/master/src/features/meeting-vote.ts https://github.com/jaoafa/jaotan.ts/blob/master/src/events/meeting-vote-reaction.ts

book000 commented 1 year ago

ひとつ、‎MeetingVote.isMultipleVote が怪しいかもしれない。 賛成票と白票同時にリアクションした場合、なんか弾かれずに動いてる気がする。

あと根本的に本当に最新のリアクション情報を持ってきてるのかが微妙な気がしてならない

book000 commented 1 year ago

calculateBorder、以下が正しいかもしれないとのこと。

Math.ceil((totalCount - whiteCount + 1) / 2)

book000 commented 1 year ago

スタンプだけなど、contentが0文字の場合にエラーで落ちる

https://github.com/jaoafa/jaotan.ts/blob/e1c5ef157b7805143b4ca288979b8742008fadaf/src/features/meeting-vote.ts#L408 IMG_3034