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

ファイルアップロード・音声認識からユースケースに「送る」ボタン #490

Merged
merged 1 commit into from
May 8, 2024

Conversation

tbrand
Copy link
Contributor

@tbrand tbrand commented May 8, 2024

#486

表題の通り実装しました。

送り元一覧

  • ファイルアップロード
  • 音声認識

送り先一覧

  • チャット
  • 文章生成
  • 要約
  • 校正
  • 翻訳

.replace(/ /g, ' ') // スペース
);
}

Copy link
Contributor Author

Choose a reason for hiding this comment

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

これがあると query parameter で指定した sentence を "その前に指定していた" sentence で上書きしちゃう (つまり、query parameter が無視されてしまう) ので、処理上問題ない上に移動しました。

setSentence(combinedTranscript);
if (combinedTranscript.length > 0) {
setSentence(combinedTranscript);
}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

これがあると初期化時に query parameter を空文字で上書きされてしまうことがあるため、空文字の際は setSentence しないようにしました

Copy link
Contributor

@wadabee wadabee left a comment

Choose a reason for hiding this comment

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

LGTM!!

@wadabee wadabee merged commit a768997 into main May 8, 2024
2 checks passed
@wadabee wadabee deleted the send-to-usecase branch May 8, 2024 01:53
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