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

localization fixes #41

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

localization fixes #41

wants to merge 2 commits into from

Conversation

AmirSoln
Copy link
Collaborator

No description provided.

@AmirSoln AmirSoln self-assigned this Nov 16, 2020
Comment on lines +37 to +39
"UNEXPECTED_ERROR": "ארעה שגיאה בלתי צפויה!",
"WRONG_CREDENTIALS": "אחד הפרטים שהכנסת שגוי!",
"UNKNOWN_ERROR": "ארעה שגיאה לא ידועה.",
Copy link
Collaborator

Choose a reason for hiding this comment

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

RemoveUNKNOWN_ERROR translation(Keep UNEXPECTED_ERROR)

Comment on lines +37 to +39
"UNEXPECTED_ERROR": "An unexpected error has occurred!",
"WRONG_CREDENTIALS": "One of the credentials is incorrect!",
"UNKNOWN_ERROR": "An unknown error has occurred.",
Copy link
Collaborator

Choose a reason for hiding this comment

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

RemoveUNKNOWN_ERROR translation(Keep UNEXPECTED_ERROR)

Comment on lines +109 to +113
// ----------------------------------------------------------------------------------------------------------------
// --TODO: is this more readable than 'CONTENT_REQUEST_SUCCESSFULLY'.tr() --
// --To auto generate code completion we need to use command: --
// --flutter pub run easy_localization:generate -f keys -o locale_keys.g.dart --source-dir ./assets/translations/--
// ----------------------------------------------------------------------------------------------------------------
Copy link
Collaborator

Choose a reason for hiding this comment

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

remove

_identitySubject.sink.addError(
NetworkError(res.statusCode, 'An unknown error has occurred.'));
_identitySubject.sink
.addError(NetworkError(res.statusCode, 'UNKNOWN_ERROR'.tr()));
Copy link
Collaborator

Choose a reason for hiding this comment

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

replace with UNEXPECTED_ERROR

Copy link
Collaborator

Choose a reason for hiding this comment

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

call with LocalKeys

season.episodes.forEach((element) {
if (element.status == MediaContentStatus.MISSING)
missingEpisodes.add(element.number);
});
}

Widget get header => Text("Season ${season.number}");
Widget get header => Text('${'SEASON'.tr()} ${season.number}');
Copy link
Collaborator

Choose a reason for hiding this comment

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

call with LocalKeys

_identitySubject.sink.addError(NetworkError(
res.statusCode, 'One of the credentials is incorrect!'));
_identitySubject.sink
.addError(NetworkError(res.statusCode, 'WRONG_CREDENTIALS'.tr()));
Copy link
Collaborator

Choose a reason for hiding this comment

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

call with LocalKeys

@@ -99,7 +100,7 @@ class _SeriesRequestSelectionState extends State<SeriesRequestPage> {
height: 30,
child: FlatButton(
child: Text(
'Select All',
'SELECT_ALL'.tr(),
Copy link
Collaborator

Choose a reason for hiding this comment

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

call with LocalKeys

@@ -115,7 +116,7 @@ class _SeriesRequestSelectionState extends State<SeriesRequestPage> {
)
: FlatButton(
child: Text(
'Submit',
'SUBMIT'.tr(),
Copy link
Collaborator

Choose a reason for hiding this comment

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

call with LocalKeys

@@ -21,7 +21,8 @@ class _SeasonExpansionTileState extends State<SeasonExpansionTile> {
return ListTile(
title: widget.item.header,
subtitle: (widget.episodeRequests.isNotEmpty)
? Text("${widget.episodeRequests.length} episodes selected")
? Text(
'${widget.episodeRequests.length} ${'EPISODES_SELECTED'.tr()}')
Copy link
Collaborator

Choose a reason for hiding this comment

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

call with LocalKeys

});
},
)
child: Text('CANCEL'.tr()),
Copy link
Collaborator

Choose a reason for hiding this comment

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

call with LocalKeys

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants