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

フィールド設定の全角ひらがなチェックに"ー”(長音符)が含まれていない #2720

Closed
rabbitchang opened this issue Sep 4, 2023 · 8 comments
Labels
Bug バグ Reviewed レビュー済 Ver4 バージョン4

Comments

@rabbitchang
Copy link

rabbitchang commented Sep 4, 2023

概要

フィールド設定の全角ひらがなチェックに"ー”(長音符)が含まれていない
理由:公文書規定では外国人名などのカナ書で長音符を含む名称などはひらがなで長音符を使っても良いことになっています

baserCMS version : 全バージョン

@ryuring ryuring added the Bug バグ label Sep 4, 2023
@ryuring
Copy link
Collaborator

ryuring commented Sep 15, 2023

@rabbitchang こちらはメールプラグインのメールフィールドですか?
カスタムコンテンツもフィールドを持っているので念のため確認です。

@ryuring ryuring added the Reviewed レビュー済 label Sep 15, 2023
@rabbitchang
Copy link
Author

@ryuring はい!気づいたのはメールフォームのテストをしている時です。

@ryuring
Copy link
Collaborator

ryuring commented Sep 15, 2023

@rabbitchang 了解です!

@ryuring
Copy link
Collaborator

ryuring commented Sep 15, 2023

@HungDV2022 メールフィールドのバリデーションを調整してください。
全角ひらがなチェックのバリデーションが入っている時に「ー」が含まれているとエラーとなっている。
「ー」が含まれている際もエラーとならないようにしてください。

「まーく」OK
「ま-く」NG

@HungDV2022
Copy link
Collaborator

@ryuring
ローカル環境でテストしてみましたが、問題なかったです。確認お願いします。
スクリーンショット 2023-09-19 9 25 14
スクリーンショット 2023-09-19 9 26 06

@HungDV2022 HungDV2022 assigned ryuring and unassigned HungDV2022 Sep 19, 2023
@ryuring
Copy link
Collaborator

ryuring commented Sep 20, 2023

@HungDV2022 確認ありがとうございました。
@rabbitchang ということは、5系では発生せず、4系のみの問題ということだと思います。
4系ラベルを貼っておきます。

@ryuring ryuring added the Ver4 バージョン4 label Sep 20, 2023
@ryuring ryuring removed their assignment Sep 20, 2023
@katokaisya
Copy link
Collaborator

@ryuring プルリク送りました。
#2787

@ryuring
Copy link
Collaborator

ryuring commented Oct 6, 2023

こちらのプルリクで解決
#2787

@ryuring ryuring closed this as completed Oct 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug バグ Reviewed レビュー済 Ver4 バージョン4
Projects
None yet
Development

No branches or pull requests

4 participants