-
Notifications
You must be signed in to change notification settings - Fork 1
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
feat: 登録削除コマンド "vcspeaker remove" の追加 #173
Conversation
book000
commented
Jul 6, 2024
- close サーバ登録解除コマンドが存在しない #12
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.
一応確認とかほしくないですか?
「すべて削除しますけどよろしいですか?戻せませんよ?」
みたいな
あと Guild ごとのデータって Alias とか Ignore とかもありますが、そこらへんは削除しませんか?
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.
まあ、たしかに。
/vcspeaker remove confirm
とか追加で叩かせるようにしますか?
Alias, Ignoreの削除については不足しているので追加します。
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.
ボタンの方が簡単かなって思ってましたがどうですか?
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.
あーそうか、テキストコマンドは無いからボタンインタラクションとか使えるのか…。
そっちのほうがいいですね
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.
確認処理と、エイリアスとかの情報削除処理を追加しました。
こういうのもテストコード書きたいですね、モックの書き方含め、よくわからんですけど…。
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.
他の Embed のタイトルと合わせました
src/main/kotlin/com/jaoafa/vcspeaker/commands/VCSpeakerCommand.kt
Outdated
Show resolved
Hide resolved
src/main/kotlin/com/jaoafa/vcspeaker/commands/VCSpeakerCommand.kt
Outdated
Show resolved
Hide resolved
src/main/kotlin/com/jaoafa/vcspeaker/commands/VCSpeakerCommand.kt
Outdated
Show resolved
Hide resolved
Co-authored-by: yuuaHP <[email protected]>