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

コマンドをGuildごとに登録し、VCSpeakerでの状態に合わせてコマンドを登録する #10

Open
book000 opened this issue Sep 24, 2023 · 13 comments
Assignees

Comments

@book000
Copy link
Member

book000 commented Sep 24, 2023

ちょっと仕様理解が追いついてなく、認識誤りだったらごめんなさい

サーバ追加手順として、以下だと認識しています。

  1. VCSpeaker.ktを起動する
  2. Botをサーバに追加する
  3. /vcspeaker settings channel:#vc でチャンネルを設定する → サーバがVCSpeakerの対象となる

このあと、Botに読み上げてもらうために指定したテキストチャンネルで発言したり、スラッシュコマンドの /join コマンドの利用を試しました。
しかし、指定したテキストチャンネルで発言しても参加→読み上げはされない、スラッシュコマンドとして join が登録されていない問題がありそうです。
(テキストコマンド $join は動作したので、スラッシュコマンドの再登録処理ができてないのかな?の認識です。)

@book000
Copy link
Member Author

book000 commented Sep 24, 2023

そもそも、サーバを対象に追加する手順として /vcspeaker settings channel:#vc で合ってるのか、わかっていません。
(チャンネル設定とかって、サーバをVCSpeakerの対象サーバと登録した後の手順かと考えていましたが、settings channel: でそのまま登録されている?)

@yuuahp
Copy link
Member

yuuahp commented Sep 25, 2023

そのまま登録されます。
/join とかは今見たら登録されてたので公開コマンドの遅延(&キャッシュ?)かと
でも読み上げられないのは謎です
あとで試します

@book000
Copy link
Member Author

book000 commented Sep 25, 2023

/join とかは今見たら登録されてたので公開コマンドの遅延(&キャッシュ?)かと

TomachiBotで動かしてたんですが、登録されてなく無いですか…?
Guild対象であれば即座に反映の筈です。

@book000
Copy link
Member Author

book000 commented Sep 25, 2023

理由無いなら無いでいいんですが、/vcspeaker server add / /vcspeaker server remove だった前回から、/vcspeaker settings channel: にした理由ってなんですか…?

正直チャンネル設定=サーバ登録ってあんまり直感的じゃなくない?って感触なのですが、何か考えてることがあれば。
もっというと、vcspeaker settingsまででしか説明が出て来ず、細かなフラグがなんの設定なのか、どうにもいまいちわからんなあという感じです

@book000
Copy link
Member Author

book000 commented Sep 25, 2023

今更言うことじゃねーよ!って感じかもですが……

@yuuahp
Copy link
Member

yuuahp commented Sep 25, 2023

/join とかは今見たら登録されてたので公開コマンドの遅延(&キャッシュ?)かと

TomachiBotで動かしてたんですが、登録されてなく無いですか…? Guild対象であれば即座に反映の筈です。

IMG_0093
Guild対象は開発時だけです

@yuuahp
Copy link
Member

yuuahp commented Sep 25, 2023

理由無いなら無いでいいんですが、/vcspeaker server add / /vcspeaker server remove だった前回から、/vcspeaker settings channel: にした理由ってなんですか…?

正直チャンネル設定=サーバ登録ってあんまり直感的じゃなくない?って感触なのですが、何か考えてることがあれば。 もっというと、vcspeaker settingsまででしか説明が出て来ず、細かなフラグがなんの設定なのか、どうにもいまいちわからんなあという感じです

それをやるなら /vcspeaker setup かなと
settings にしたのはどうせ身内鯖でしか使わないしオンボーディングとか要らないかなと思ったためです
必要なら実装します
あとオプションの説明は書いたはずですが見れませんか
今はとりあえず動かす段階なので許して…

@book000
Copy link
Member Author

book000 commented Sep 25, 2023

あーー、スラッシュコマンド登録は全サーバ向けのやつではなく、Guild対象の方を強くお勧めさせてください。

というのは、マジで全サーバ向け登録は時間がかかるのと、サーバ毎に有効化するコマンドをマネジメントできない =「サーバ登録できてない(/vcspeaker settings channel: してない)のに、joinコマンドが叩ける」というように、ユーザにとって使いにくくなりがちかなと思っています。

まあ、登録サーバだけ使えるコマンドとかいう話は後々でもいいですが全サーバ展開のやつはマジで遅いので全くお勧めできないです…

@book000
Copy link
Member Author

book000 commented Sep 25, 2023

IMG_2983
IMG_2984

ここのニュアンスで、サーバ登録ってニュアンスが何もわからんなあと思ったりしていました

まあ、VCSpeakerそのものをセットアップするのはどうせ私になるので(えくさす鯖とか他所であっても)いいっちゃいいんですが、自分が作った物ですらあとから使って使い方を忘れるので、他の人が作った物を使うのも尚更忘れると思うんですよね
だったらストンとわかりやすい形が望ましいかな?と思っていましたが…まあ、作ってもらってる手前色々言えた話でも無いなあとは思っているので、即今どうこうしろとは言えないです

今後、私がコードいじるときにいじるかもしれないけど…


まあ、チャンネル登録 = サーバ登録の設計ならそれはそれで良いのですが、サーバ登録解除の設計はどういう風にする見込みですか?
channel:null とかになってしまいそうな…

@yuuahp
Copy link
Member

yuuahp commented Sep 25, 2023

今のところ解除の方法はありません(致命的)
なるはやでどうにかします

@book000
Copy link
Member Author

book000 commented Sep 25, 2023

😭

@yuuahp
Copy link
Member

yuuahp commented Sep 25, 2023

今すぐどうしてもっていうなら store/guilds.json を消せば解除されます

@book000 book000 changed the title vcspeaker settings channel: で設定した後、joinコマンドがサジェストされない コマンドをGuildごとに登録し、VCSpeakerでの状態に合わせてコマンドを登録する Sep 26, 2023
@yuuahp
Copy link
Member

yuuahp commented Sep 26, 2023

メモ: ExtensibleBot.load("extension") / .unload("extension")

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

No branches or pull requests

2 participants