Skip to content

Conversation

@masaton0216
Copy link
Contributor

@masaton0216 masaton0216 commented Apr 19, 2024

概要

  • クライアントから指摘あり。より明示的で次の所作がわかりやすい為、そのまま採用します。
画像認証画面で、入力をミスした場合に表示されるエラーメッセージを、

 画像認証には正しい画像の文字を指定してください。
→ 画像上に表示されている正しい文字を入力してください。

のようにご修正いただくことはできますでしょうか。
(※「指定」ではなく「入力」としていただきたい点と、
  「正しい」がかかる言葉を、近くに置きたい、という主旨でのご依頼です。)

レビュー完了希望日

なし

関連Pull requests/Issues

なし

参考(対応後イメージ)

image

DB変更の有無

なし

チェックリスト

@masaton0216 masaton0216 added the enhancement 機能強化 label Apr 19, 2024
@masaton0216 masaton0216 self-assigned this Apr 19, 2024
@masaton0216 masaton0216 merged commit b9cf679 into master Apr 19, 2024
@masaton0216 masaton0216 deleted the fixed_capture_validation_message branch April 19, 2024 01:21
Copy link
Contributor

@akagane99 akagane99 left a comment

Choose a reason for hiding this comment

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

ここの修正どうだろう?

{
// カスタムエラーメッセージの定義
$validation_messages = [
'captcha.captcha' => '画像上に表示されている正しい文字を入力してください。',
Copy link
Contributor

Choose a reason for hiding this comment

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

https://github.com/opensource-workshop/connect-cms/blob/master/resources/lang/ja/validation.php#L175

langの英日を変えちゃっていいと思うけど、どうだろう?

Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ありがとうございます!
langの存在をすっかり忘れておりました(笑)
別PR飛ばします。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement 機能強化

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants