Skip to content
This repository was archived by the owner on Jun 13, 2024. It is now read-only.

Modify the deviceLocale workaround for all desktop platforms #187

Merged
merged 3 commits into from
Jun 19, 2020

Conversation

guidezpl
Copy link
Member

@guidezpl guidezpl commented Jun 18, 2020

Workaround currently only accounts for macOS when it should account for all desktop platforms.
Also update ios/Podfile, android/settings.gradle and other files for smooth running on all platforms (autogenerated)

Closes #172

Copy link
Contributor

@rami-a rami-a left a comment

Choose a reason for hiding this comment

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

LGTM

@@ -1,7 +1,3 @@
// Copyright 2014 The Flutter Authors. All rights reserved.
Copy link
Contributor

Choose a reason for hiding this comment

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

Why remove the copyright header?

Copy link
Member Author

Choose a reason for hiding this comment

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

This file was autogenerated, so leaving as is

@guidezpl guidezpl merged commit 576217a into master Jun 19, 2020
@guidezpl guidezpl deleted the fix-devicelocale-workaround branch June 19, 2020 13:42
guidezpl added a commit that referenced this pull request Jun 27, 2020
* Update gallery_options.dart

* Update gallery_options.dart

* Update Podfile & android settings.gradle

Former-commit-id: 576217a
DanTup pushed a commit to DanTup/flutter_gallery that referenced this pull request Jun 30, 2020
Add details for the Shrine study in the Gallery app:
Use correct letter spacing from the spec.

Former-commit-id: d5189f8
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

flutter: Invalid argument(s): Invalid locale "null"
2 participants