search

k-seta / trpg-bot

MIT License
1 stars 0 forks source link

マヨキン用txtファイルをzipでDL #31

Closed harryusei closed 4 years ago

harryusei commented 4 years ago

25 の対応

DropboxLogic.sync()を更新しました． 以下の通りに動作します．

  1. ./trpg_bot/resources内にmayokinディレクトリがある場合，それを削除
  2. dropbox.files_download_zip_to_fileで./trpg_bot/resourcesにzipファイルをDL
  3. zipファイルを解凍し，./trpg_bot/resources/mayokin内にtxtファイル群が配置される
  4. zipファイルを削除
aknorsh commented 4 years ago

コマンド受付時にも一回printしたいと思ったんですがいかがでしょう。typoしたのか応答待ちか分からない瞬間があるのがちょっと嫌かなと思いました。

harryusei commented 4 years ago

コマンド受付時にも一回printしたいと思ったんですがいかがでしょう。typoしたのか応答待ちか分からない瞬間があるのがちょっと嫌かなと思いました。

/syncコマンド受付時，botに発言はさせなくていいですか？ コマンド終了時にはbotが喋るので，そっちに統一したほうがいいかなと思ったのですがどうでしょう

aknorsh commented 4 years ago

すいません、文脈を読み間違えて「コマンド受付時にもbotにmessageを送らせててはいかがでしょうか」という意味でprintするのはどうでしょう？と言っておりました。

/syncコマンド受付時，botに発言はさせなくていいですか？

むしろこっちを実施して、print関数は処理終了時1回でよいと思います。

harryusei commented 4 years ago

すいません、文脈を読み間違えて「コマンド受付時にもbotにmessageを送らせててはいかがでしょうか」という意味でprintするのはどうでしょう？と言っておりました。

/syncコマンド受付時，botに発言はさせなくていいですか？

むしろこっちを実施して、print関数は処理終了時1回でよいと思います。

0580d78 で対応しました．それらしく喋らせてみました

harryusei commented 4 years ago

レビューありがとうございました！