Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the
✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codecov Report❌ Patch coverage is
❌ Your patch status has failed because the patch coverage (3.33%) is below the target coverage (80.00%). You can increase the patch coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## main-design-system #41 +/- ##
=====================================================
Coverage ? 43.49%
=====================================================
Files ? 75
Lines ? 1876
Branches ? 0
=====================================================
Hits ? 816
Misses ? 1060
Partials ? 0 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
# Conflicts: # apps/design_system_gallery/lib/app/gallery_app.directories.g.dart # packages/stream_core_flutter/lib/src/factory/stream_component_factory.dart # packages/stream_core_flutter/pubspec.yaml
…/filetype-icons # Conflicts: # packages/stream_core_flutter/pubspec.yaml
Submit a pull request
Linear: FLU-368
CLA
Description of the pull request
This adds the icons for file types: https://www.figma.com/design/Us73erK1xFNcB5EH3hyq6Y/Chat-SDK-Design-System?node-id=157-2400&m=dev
As per discussion on Figma, the icons are simplified compared to chat, the text is now variable so we can have a limited set of svg's.
I have all mime types from chat: https://github.com/GetStream/stream-chat-flutter/blob/master/packages/stream_chat_flutter/lib/src/utils/helpers.dart#L334-L388
And added video types.
Screenshots / Videos