Merged
Conversation
メッセージID精査に伴ってテスト修正
kiy0taka
approved these changes
Aug 16, 2018
nanasess
requested changes
Aug 16, 2018
| @@ -38,7 +38,7 @@ class ShopController extends AbstractController | |||
| /** | |||
| * @var BaseInfo | |||
Contributor
There was a problem hiding this comment.
BaseInfoRepository に変更をお願いします
nanasess
approved these changes
Aug 16, 2018
okazy
approved these changes
Aug 16, 2018
Merged
Merged
Closed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
メッセージIDの精査および関する不具合の修正を行っています。
命名規約
原則、以下のフォーマットで作成しています。
[prefix].[機能または画面].[操作や文言を表す文字列]prefix
同じ意味だが異なる文言が存在するため, フロント画面と管理画面ではprefixをつけ
たとえば、
ご注文(フロント) / 受注 (管理画面)などの文言です。Prefixは以下のとおりです。
区切り文字
namespaceには
.を使用します。単語の区切り文字としては
_を使用しています。また、同一の意味で表示箇所により文言が変わるものや、類義語には
__を使用しています。たとえば
商品画像は、商品登録画面では商品画像、商品一覧画面では画像と省略されるため、以下のようにメッセージIDをつけています。namespaceの粒度
namespaceの区切りは、細かい粒度でつけると重複が多く発生するため、機能単位レベルで区切るほうが命名しやすいです。
メッセージファイル
メッセージファイルは、従来のPHPファイルからyamlファイルに変更しました。
messages.ja.yaml, validators.ja.yamlとなります。その他
実装中等の理由により、以下の画面は対応していません。
ツールチップや、フォームタイプのエラーメッセージは本PRでは対応していません。