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

Badボタンとフィードバックの充実 #660

Merged
merged 4 commits into from
Sep 24, 2024

Conversation

yuukiok
Copy link
Collaborator

@yuukiok yuukiok commented Sep 20, 2024

Issue #, if available:
655

Description of changes:

Badボタンを押した際に、フィードバックフォームが出現する。

フィードバックフォーム送信前にbadの情報はDBに送信される。(フィードバックフォームをキャンセルしてもbadが消えないようにするため)
image

送信した場合

DBに情報を登録・3秒間Thank youコメントを表示。
image

DynamoDB

以下二つが追加される

  • reasons: 選択肢の配列
  • detailedFeedback: フィードバックコメント
image

フィードバックフォームを送信せずキャンセルした場合

上述しているようbadは消えない。
image

@wadabee wadabee self-requested a review September 20, 2024 06:39
const [feedback, setFeedback] = useState<string>('');
const [error, setError] = useState<string>('');

const reasons = ['不正確', '時期遅れ', '有害または攻撃的', 'その他'];
Copy link
Contributor

Choose a reason for hiding this comment

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

時期遅れというのは、情報が古い、ということでしょうか?
わかりやすく言い換えていただけると助かります!

onClick={handleSubmit}
>
送信
</button>
Copy link
Contributor

Choose a reason for hiding this comment

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

button はスタイルを統一したいので components/Button.tsx を使っていただくことは可能でしょうか?

Copy link
Contributor

@tbrand tbrand left a comment

Choose a reason for hiding this comment

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

ありがとうございます!LGTMです!

@tbrand tbrand merged commit 12b8a2a into aws-samples:main Sep 24, 2024
2 checks passed
@yuukiok yuukiok self-assigned this Sep 30, 2024
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.

2 participants