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

多言語対応 #43

Closed
wants to merge 9 commits into from
Closed

多言語対応 #43

wants to merge 9 commits into from

Conversation

iseruuuuu
Copy link
Contributor

@iseruuuuu iseruuuuu commented Jan 5, 2024

概要

  • 多言語対応を行いました。

  • マルチパッケージ構成でも使いやすいようにする対応は別PRで実装予定です。

  • close 多言語対応 #3

タスク

  • 多言語対応に必要なPackageの追加する。
  • arbファイルを日本語と英語の2種類作成する。
  • l10n.yamlに必要なコードを記載する。
  • コード生成を行う。
  • MaterialAppに必要な記載を行う。
  • main.dartとは別のファイル(HomeScreen)のAppBarにタイトルを仮で追記する 。

レビューする上で伝えておきたいこと

レビュー観点

レビューレベル

  • Lv1: ぱっとみて違和感がないかチェックして Approve する
  • Lv2: 仕様レベルまで理解して、仕様通りに動くかある程度検証して Approve する
  • Lv3: 実際に環境で動作確認したうえで Approve する

レビュー優先度

  • すぐに見てもらいたい ( hotfix など ) 🚀
  • 今日中に見てもらいたい 🚗
  • 今日〜明日中で見てもらいたい 🚶
  • 数日以内で見てもらいたい 🐢

画像 / 動画

日本語設定 英語設定

動作確認手順

備考

Copy link

github-actions bot commented Jan 5, 2024

Ready for review 🚀

@iseruuuuu iseruuuuu added the enhancement New feature or request label Jan 5, 2024
@iseruuuuu iseruuuuu marked this pull request as ready for review January 9, 2024 00:43
Copy link

github-actions bot commented Jan 9, 2024

Ready for review 🚀

Comment on lines +1 to +11
// ignore_for_file: type=lint

import 'l10n.dart';

/// The translations for English (`en`).
class L10nEn extends L10n {
L10nEn([String locale = 'en']) : super(locale);

@override
String get appBar => 'Flutter Template';
}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

[メモ]
この辺りの生成ファイルは.gitattributesを作成して、追記してもいいかもしれないなと思いました。
ご意見をぜひいただけると嬉しいです!

@iseruuuuu
Copy link
Contributor Author

@Yamasaki-pan961 @warahiko
お手隙の際にレビューお願いします🙇

@iseruuuuu iseruuuuu changed the title 【WIP】多言語対応 多言語対応 Jan 9, 2024
# Conflicts:
#	apps/app/lib/main.dart
@iseruuuuu
Copy link
Contributor Author

こちらのPRも署名Commitができていなかったので、再度作り直します

@iseruuuuu iseruuuuu marked this pull request as draft January 10, 2024 07:16
@iseruuuuu
Copy link
Contributor Author

#48
こちらで署名Commitをしたので、こちらのPRはCloseさせていただきます

@iseruuuuu iseruuuuu closed this Jan 10, 2024
@blendthink blendthink deleted the feature/add_l10n branch January 29, 2024 06:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

多言語対応
1 participant