-
-
Notifications
You must be signed in to change notification settings - Fork 170
Add support Telegram Sticker .tgs
in Lottie Integration
#659
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
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #659 +/- ##
=======================================
Coverage 97.55% 97.55%
=======================================
Files 21 21
Lines 819 819
=======================================
Hits 799 799
Misses 20 20 ☔ View full report in Codecov by Sentry. |
da1a9cf
to
5f3c681
Compare
@AlexV525 I noticed a problem when generating the example files because it was changing the format code, I rolled back to old example code and added changed code lines with my added assets |
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.
Thanks!
## 5.10.0 **Feature** - [#659](#659) Add support Telegram Sticker `.tgs` in Lottie Integration. by [@dungngminh](https://github.com/dungngminh) **Bug fix** - [#653](#653) Constraints `dart_style` to `>=2.3.7`. by [@AlexV525](https://github.com/AlexV525) - [#656](#656) Add missing parameters in Lottie integration from lottie 3.0.0. by [@dungngminh](https://github.com/dungngminh) - [#658](#658) Update required dart version. by [@koji-1009](https://github.com/koji-1009)
## 5.10.0 **Feature** - [#659](FlutterGen/flutter_gen#659) Add support Telegram Sticker `.tgs` in Lottie Integration. by [@dungngminh](https://github.com/dungngminh) **Bug fix** - [#653](FlutterGen/flutter_gen#653) Constraints `dart_style` to `>=2.3.7`. by [@AlexV525](https://github.com/AlexV525) - [#656](FlutterGen/flutter_gen#656) Add missing parameters in Lottie integration from lottie 3.0.0. by [@dungngminh](https://github.com/dungngminh) - [#658](FlutterGen/flutter_gen#658) Update required dart version. by [@koji-1009](https://github.com/koji-1009)
## What does this change? Generated files are being formatted based on confusing conditions. The PR refactored how we construct the Dart formatter in a more elegant manner. 1. The formatter will try to find a valid Dart SDK constraint from `pubspec.yaml` / `pubspec.lock`. Otherwise, it will fall back to using the current Dart SDK version as the constraint. The behavior is like: 1. When the user is using Dart SDK 3.8.0, `pubspec.yaml` has `^3.4.0`, and `pubspec.lock` resolves `^3.7.0`, the formatter will format like 3.4.0. 2. When the runner is being used for testing purposes, which the specified pubspec file does not contain `environment - sdk` (and likely does not have a `pubspec.lock`), the formatter will format based on the current Dart SDK version. 2. The formatter will try to find a valid page width from `analysis_options.yaml - formatter - page_width` / `pubspec.yaml - flutter_gen - line_length`, then prepend the format width header to make all format tools respect the config. - Fixes #685 (comment) - Fixes #659 (comment) - Fixes #620 ## Type of change - [x] New feature (non-breaking change which adds functionality)
What does this change?
.tgs
files can be loaded by providing a special decoder. This MR will support.tgs
.Screen.Recording.2025-02-14.at.20.50.07.mp4
Type of change
Please delete options that are not relevant.
Checklist:
Before submitting your PR, there are a few things you can do to make sure it goes smoothly:
melos run test
)melos run format
to automatically apply formatting)