aws-samples / bedrock-claude-chat

AWS-native chatbot using Bedrock + Claude (+Mistral)
MIT No Attribution
687 stars 230 forks source link

fix message #375

Closed k70suK3-k06a7ash1 closed 5 days ago

k70suK3-k06a7ash1 commented 2 weeks ago

Issue #, if available:

338

Description of changes:

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

k70suK3-k06a7ash1 commented 2 weeks ago

@statefb こちら修正しました!

k70suK3-k06a7ash1 commented 1 week ago

@statefb 修正しました! 多分大丈夫かと・・・

README.mdの方に

- After about 30 minutes, you will get the following output, which you can access from your browser

がありましたが、おそらく文脈的につながらないように見えたのでen,jaともに該当箇所を削除してますmm

k70suK3-k06a7ash1 commented 1 week ago

でも、bash.shの実行後に表示される部分を以下〇〇、として記載した方が良いような気もしてきました・・・

パラメータを指定したコマンド例

の部分の出力結果に近しいものが出力されるかなと思いますので、如何しましょう?

k70suK3-k06a7ash1 commented 1 week ago

デプロイ時に以下のパラメータを指定することで、セキュリティとカスタマイズを強化でき「ます」に修正いただけますか?

こちら、承知です!

statefb commented 6 days ago

@k70suK3-k06a7ash1 ありがとうございます!改めて見直していて思ったのですが、

以上を踏まえると、元のy/Nに戻すべきかなーと考えています。元々のorがわかりにくい件については、下記のようにすれば回避可能かなと思っています。

Were you using the old versions v0.x or prior, or are you a new user starting with v1.x or later? (y/N):

もちろん、これまで手を加えていただいたREADMEの整理はそのままマージさせていただければありがたいです!
(色々と手を加えていただいたにも関わらず申し訳ないですmm)

k70suK3-k06a7ash1 commented 6 days ago

@statefb

重要な意思決定選択である一方、既にリリース済みのため論理反転変更することにリスクがある

この点、確かにおっしゃる通りですね・・・ 元に戻す方針、賛成です!

k70suK3-k06a7ash1 commented 6 days ago

@statefb 多分、このPRを修正するより、ゼロから作成して対応した方が影響範囲狭い気がしたので、このままCloseいただいても全然構いませんので! (お気遣いいただいてたらと思い、念の為・・・)

statefb commented 6 days ago

@k70suK3-k06a7ash1 承知です!確かに新規で作り直した方が早いかもしれないですね。ありがとうございます!

statefb commented 5 days ago

duplicated #402