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

Update to the latest localizations - Adds missing DatePicker strings and fixes manually entering date for German #63026

Merged
merged 9 commits into from
Aug 6, 2020

Conversation

shihaohong
Copy link
Contributor

@shihaohong shihaohong commented Aug 5, 2020

Description

Continues on #62896, since @rami-a may have internet connectivity issues over the next day or so.

Updates the translated strings in the Flutter project.

Oddness with _nb files:

Edit: It seems that the internal translation console now generates both _nb.arb and _no.arb files. In that case, we should keep both versions to keep things consistent with the translation console. Given this change, we should be able to remove the workaround that makes _no.arb and _nb.arb synonyms in the framework code, but I'll do it in a separate PR to make sure that after removing the code, everything continues to work as expected.

Somehow, _nb.arb files are now also added by Google's internal translation console alongside _no.arb. I'll need to see if this is expected, as it may be a new update to how these are generated (we currently generate _nb.dart source files through duplicating _no.arb's messages, so it's quite odd that _nb.arb files are being generated). Another oddity is that the _nb.arb translations seem incomplete on the translation console. For this PR, I just manually removed the _nb.arb source files, but this inconsistency needs to be investigated.

Related Issues

Fixes #55535

Tests

I added the following tests:

  • n/a, since it regenerates translations from a script

Checklist

Before you create this PR, confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • I signed the CLA.
  • I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
  • I read the Tree Hygiene wiki page, which explains my responsibilities.
  • I updated/added relevant documentation (doc comments with ///).
  • All existing and new tests are passing.
  • The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
  • I am willing to follow-up on review comments in a timely manner.

Breaking Change

Did any tests fail when you ran them? Please read Handling breaking changes.

@flutter-dashboard flutter-dashboard bot added a: internationalization Supporting other languages or locales. (aka i18n) f: material design flutter/packages/flutter/material repository. labels Aug 5, 2020
@flutter-dashboard
Copy link

It looks like this pull request may not have tests. Please make sure to add tests before merging. If you need an exemption to this rule, contact Hixie on the #hackers channel in Chat.

Reviewers: Read the Tree Hygiene page and make sure this patch meets those guidelines before LGTMing.

@shihaohong shihaohong added f: cupertino flutter/packages/flutter/cupertino repository framework flutter/packages/flutter repository. See also f: labels. labels Aug 5, 2020
Copy link
Contributor

@HansMuller HansMuller left a comment

Choose a reason for hiding this comment

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

LGTM without the _nb stuff for now.

@shihaohong
Copy link
Contributor Author

Removed _nb.arb and source Dart files for now, per Hans's suggestion

@fluttergithubbot fluttergithubbot merged commit bd3a4cc into flutter:master Aug 6, 2020
@shihaohong shihaohong mentioned this pull request Aug 6, 2020
13 tasks
@shihaohong shihaohong changed the title Update to the latest localizations Update to the latest localizations - Adds missing DatePicker strings and fixes manually entering date for German Aug 10, 2020
@pcsosinski
Copy link

removing cp label in favor of #55535

pcsosinski pushed a commit that referenced this pull request Aug 13, 2020
* Fix SliverList scrollOffsetCorrection 0 case (#62615)

* Case insensitive check flavor names against Xcode schemes (#61140)

* Address misc time picker design issues (#62803)

* Update to the latest localizations (#63026)

* Removed the inputFormatters from the text input fields used by the Date Pickers (#63461)

* Fix App.framework path in Podfile (#63412)

* Update engine hash to 1.20.2

Co-authored-by: Kate Lovett <katelovett@google.com>
Co-authored-by: Jenn Magder <magder@google.com>
Co-authored-by: Rami <2364772+rami-a@users.noreply.github.com>
Co-authored-by: Shi-Hao Hong <shihaohong@google.com>
Co-authored-by: Darren Austin <darrenaustin@google.com>
mingwandroid pushed a commit to mingwandroid/flutter that referenced this pull request Sep 6, 2020
@shihaohong shihaohong deleted the de_localizations branch December 3, 2020 11:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a: internationalization Supporting other languages or locales. (aka i18n) f: cupertino flutter/packages/flutter/cupertino repository f: material design flutter/packages/flutter/material repository. framework flutter/packages/flutter repository. See also f: labels.
Projects
None yet
4 participants