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.

Sign up for GitHub

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

Jump to bottom

[BUG] Question format #338

Closed
joopdo opened this issue Jun 3, 2024 · 4 comments · Fixed by #402
Closed

[BUG] Question format #338

joopdo opened this issue Jun 3, 2024 · 4 comments · Fixed by #402
Labels
needs-info Need more information such as reproduction step

Comments

@joopdo
Copy link

joopdo commented Jun 3, 2024

Running bin.sh gives "Are you a new user or already using v1.x of Bedrock Claude Chat? (y/N): "

The 'or' format of the question makes a Y/N answer confusing.
@statefb
Copy link
Contributor

statefb commented Jun 4, 2024

Thank you for pointing out! Could you take a trouble to make a pull request?

@statefb statefb added the needs-info Need more information such as reproduction step label Jun 6, 2024
@k70suK3-k06a7ash1
Copy link
Contributor

@joopdo
Do you feel it would be easier to understand if the decision is whether or not the User is using V0.x?

If you want, I can create a PR for you.

@k70suK3-k06a7ash1 k70suK3-k06a7ash1 mentioned this issue Jun 14, 2024
Closed
@k70suK3-k06a7ash1
Copy link
Contributor

@joopdo
like this
#375

please check it

@joopdo
Copy link
Author

joopdo commented Jun 14, 2024

Thanks @k70suK3-k06a7ash1 ! Much appreciated

@joopdo joopdo closed this as completed Jun 14, 2024
@statefb statefb mentioned this issue Jun 25, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
needs-info Need more information such as reproduction step
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix easy deploy message aws-samples/bedrock-claude-chat
3 participants
@joopdo @statefb @k70suK3-k06a7ash1