-
Notifications
You must be signed in to change notification settings - Fork 4
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
[#197] Keep the template project up to date with new configurations, versions from the Flutter starter project #201
[#197] Keep the template project up to date with new configurations, versions from the Flutter starter project #201
Conversation
9b2726a
to
8185136
Compare
f7b3d23
to
247beeb
Compare
bricks/template/__brick__/{{project_name.snakeCase()}}/android/app/build.gradle
Show resolved
Hide resolved
3f6413b
to
0c8c863
Compare
4c7039a
to
91f3d80
Compare
7b45227
to
18ba9fa
Compare
flutter_config seems back to work properly in version 2.0.2 https://github.com/ByneappLLC/flutter_config/commits/master 🤔 |
…s not work on the newest Gradle version"
…ermission_handler dependency
@doannimble I answered your concern here. |
bricks/template/__brick__/{{project_name.snakeCase()}}/ios/Podfile.lock
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM now 🚀
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@luongvo I'm curious do we have a plan to do this with the sample folder? 🤔
@chornerman We have this workflow https://github.com/nimblehq/flutter-templates/blob/develop/.github/workflows/generate_sample.yml. However, I plan to update the sample on each PR instead with this RFC #219 😄 |
@luongvo I see now, thanks for the explanation 😄 |
bricks/template/__brick__/{{project_name.snakeCase()}}/lib/main.dart
Outdated
Show resolved
Hide resolved
@luongvo Just a curious question, should we update the |
@doannimble I have the same answer here, please check it out #201 (comment) 👋 |
POW is updated (without Material3 enabled). |
@manh-t @sleepylee your final reviews are appreciated 🤗 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM now 🙆♂️
What happened 👀
MinimumOSVersion
to 11production
andstaging
schemes. Now we have 4 build configurations for each:project.pbxproj
to latest project structure.cupertino_icons
AdduseMaterial3: true,
to baseThemeData
inmain.dart
Insight 📝
flutter_config does not work with the newest Gradle version. While waiting for the official support, we have to switch to the alternative https://pub.dev/packages/flutter_config_plus ✅ . Updating to
flutter_config_plus
requires:minSdkVersion 21
👉23
MinimumOSVersion
to 11After updating
permission_handler: ^10.2.0
, we must run this command to let Mason re-generates the bundle file ✅Proof Of Work 📹