improve: 辞書ファイルの接頭辞を削除 #284
Conversation
github-actions
bot
added
@apps/app
yumemi-team-review-requester
bot
requested review from
a team,
rizumita and
K9i-0
and removed request for
a team
|
Ready for review 🚀 |
apps/app/lib/l10n/ja.arb
Outdated
| { | ||
| "@@locale": "ja", |
There was a problem hiding this comment.
そのままだと、ファイル名から言語を推定できず、以下のエラーがでていました。
Make sure that the locale is specified in the file's '@@locale' property or as part of the filename (e.g. file_en.arb)
回避するために、明示的に言語名を記述しています
There was a problem hiding this comment.
@mj-hd
あー、、 xx_ja.arb とかにしないとこの指定が必要になってくるのですね。。
こうなると、新しく作成する度にこの指定を追記しないといけないのですね。。
この対応についてはご相談させてください 🙏
There was a problem hiding this comment.
app_ja.arb も一般的で良さそうですね。
実は元の形でもファイル検索もしやすいですし、名前が長いだけで短く変更するメリットそこまでなかったりしないでしょうか？
There was a problem hiding this comment.
実は元の形でもファイル検索もしやすいですし、名前が長いだけで短く変更するメリットそこまでなかったりしないでしょうか？
ファイル検索のことを全然考慮できていませんでした、、！
たしかにファイルを探す時にパッケージ名で検索できると良さそうですね。
となると、命名規則を整理して修正していただくほうが良さそうですね、、！
以下のようにするのはいかがでしょうか？
- apps/app
app_ja.arb
- cores
cores_{{core_name.snakeCase()}}_ja.arb
- features
features_{{feature_name.snakeCase()}}_ja.arb
There was a problem hiding this comment.
ありがとうございます！そちらで修正してみます
|
Visit the preview URL for this PR (updated for commit 8d0f5da): https://flutter-mobile-project-template-catalog--pr284-improve-kq6uacr0.web.app (expires Sun, 28 Jul 2024 22:33:20 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 9ea56735a63d07a7cfe62eb204b0528284c37c23 |
概要
app_ja.arbなど)の接頭辞を消して、ja.arbなど短くしましたレビュー観点
レビューレベル
レビュー優先度
画像 / 動画
確認したこと
動作確認手順
備考