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

TW-1781: upgrade Flutter tot 3.22 #1790

Merged

Conversation

sherlockvn
Copy link
Contributor

@sherlockvn sherlockvn commented May 21, 2024

Change apply:

  • remove background and onBackground color, now use only surface and onSurface
  • change name from surfaceVariant => surfaceContainerHighest, MaterialStateProperty => WidgetStateProperty, MaterialState => WidgetState, use super.key for shorter form (new lint rule)
  • Migration from RawKeyEvent to KeyEvent. Read more (in conclusion, add ignore_deprecated, because that when i test it again, the up/down not work)
  • Upgrade flutter_local_notification from requestPermission => requestNotificationsPermission Changelog, Readmore
  • Upgrade url_laucher, change from Uri to WebUri, remove ChromeSafariBrowserSettings in web
  • Upgrade the index.html file in web folder
  • Upgrade other packages in pubspec.yaml to resolve conflicts

Some note:

  • Web assembly can't be build yet(a lot of packages have not supported that), web assembly not support safari + firefox yet. But the web initialization is faster, i will update an video demo soon. Read more

Testing:

IOS:

RPReplay_Final1716361592.MP4

Android above 13 new notification permission:

Screen.Recording.2024-05-23.at.06.03.05.mov

Android under 13 :

Screen.Recording.2024-05-23.at.06.08.23.mov

Web chrome:

Screen.Recording.2024-05-23.at.09.45.11.mov

Web safari:

Screen.Recording.2024-05-23.at.09.53.54.mov

Web firefox:

Screen.Recording.2024-05-23.at.09.56.55.mov

@sherlockvn sherlockvn force-pushed the upgrade_3_22/part-I-update-version-of-packages-in-pubspec branch from 69f73c0 to 832cd6c Compare May 22, 2024 06:37
@sherlockvn sherlockvn force-pushed the upgrade_3_22/part-I-update-version-of-packages-in-pubspec branch from 832cd6c to 66cd62d Compare May 22, 2024 07:25
@sherlockvn sherlockvn force-pushed the upgrade_3_22/part-I-update-version-of-packages-in-pubspec branch from 6ce5304 to 134386d Compare May 22, 2024 08:30
@sherlockvn sherlockvn force-pushed the upgrade_3_22/part-I-update-version-of-packages-in-pubspec branch from 134386d to ce76490 Compare May 22, 2024 09:20
@sherlockvn sherlockvn force-pushed the upgrade_3_22/part-I-update-version-of-packages-in-pubspec branch from ce76490 to 07adff0 Compare May 22, 2024 09:27
Copy link

This PR has been deployed to https://linagora.github.io/twake-on-matrix/1790

@sherlockvn sherlockvn force-pushed the upgrade_3_22/part-I-update-version-of-packages-in-pubspec branch from 07adff0 to 761aa6b Compare May 23, 2024 01:35
@sherlockvn sherlockvn force-pushed the upgrade_3_22/part-I-update-version-of-packages-in-pubspec branch from 761aa6b to b2fa8a2 Compare May 23, 2024 08:48
@@ -32,6 +32,7 @@
<uses-permission android:name="android.permission.READ_MEDIA_VIDEO" />
<uses-permission android:name="android.permission.READ_MEDIA_AUDIO" />
<uses-permission android:name="android.permission.ACCESS_MEDIA_LOCATION" />
<uses-permission android:name="android.permission.POST_NOTIFICATIONS"/>
Copy link
Member

Choose a reason for hiding this comment

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

please add adr

Copy link
Member

Choose a reason for hiding this comment

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

Demo this case, and compare with other application in case of Don't allow in android 13

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Screen.Recording.2024-05-24.at.21.18.34.mov

lib/config/themes.dart Outdated Show resolved Hide resolved
@hoangdat hoangdat force-pushed the upgrade_3_22/part-I-update-version-of-packages-in-pubspec branch from 782a2d8 to b531698 Compare May 25, 2024 07:26
@hoangdat hoangdat force-pushed the upgrade_3_22/part-I-update-version-of-packages-in-pubspec branch from b531698 to 113b8c5 Compare May 25, 2024 07:58
@sherlockvn sherlockvn force-pushed the upgrade_3_22/part-I-update-version-of-packages-in-pubspec branch from 113b8c5 to 839e530 Compare May 25, 2024 08:08
@hoangdat hoangdat force-pushed the upgrade_3_22/part-I-update-version-of-packages-in-pubspec branch from 839e530 to 113b8c5 Compare May 25, 2024 08:10
@sherlockvn sherlockvn force-pushed the upgrade_3_22/part-I-update-version-of-packages-in-pubspec branch from 113b8c5 to a1f8905 Compare May 25, 2024 08:12
@hoangdat hoangdat merged commit 9790230 into main May 25, 2024
12 checks passed
@hoangdat hoangdat deleted the upgrade_3_22/part-I-update-version-of-packages-in-pubspec branch May 25, 2024 10:00
@sherlockvn sherlockvn changed the title TW-1781: upgrade package inside the pubspec to compatibality with flu… TW-1781: upgrade package inside the pubspec to compatibality with flutter Jul 1, 2024
@sherlockvn sherlockvn changed the title TW-1781: upgrade package inside the pubspec to compatibality with flutter TW-1781: upgrade Flutter tot 3.22 Jul 1, 2024
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