Skip to content

Conversation

@ShuzoN
Copy link
Owner

@ShuzoN ShuzoN commented May 10, 2022

issue: #84
依存PR: #85
依存PRとの差分: refactor_argBody...bye_zenkaku_space

●やったこと

  • 全角スペースを含む文字列が渡された場合はrejectする
  • テストコードの追加

@ShuzoN
Copy link
Owner Author

ShuzoN commented May 10, 2022

@IkezoeMakoto @mii288

@ShuzoN
Copy link
Owner Author

ShuzoN commented May 10, 2022

image

Copy link
Collaborator

@mii288 mii288 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Slackのblockメッセージ内にErrorオブジェクトがそのまま使用されている(blocks.text.textの値)んですが、InvalidなJSONとしてSlack側で解釈されそうだと思いました。
この点は問題ないでしょうか?

if (args === undefined) {
if (body.text === undefined) {
throw new Error(
"引数が正しく渡されていません。もう一度helpを見てください。"
Copy link
Collaborator

@mii288 mii288 May 10, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

今回のPRとはちょっと関係ないですが、ここに到達するケースはどういうケースでしょう・・?

image

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

Successfully merging this pull request may close these issues.

3 participants