AppFlowy icon indicating copy to clipboard operation
AppFlowy copied to clipboard

[WIP] feat: media type option

Open Xazin opened this issue 1 year ago • 2 comments

Todo

  • [ ] Localization
  • [ ] Filter bug
  • [ ] Integration tests
  • [ ] Rewrite cell styles, should be reusable across all platforms

Feature Preview


PR Checklist

  • [ ] My code adheres to AppFlowy's Conventions
  • [ ] I've listed at least one issue that this PR fixes in the description above.
  • [ ] I've added a test(s) to validate changes in this PR, or this PR only contains semantic changes.
  • [ ] All existing tests are passing.

Xazin avatar Aug 27 '24 14:08 Xazin

🥷 Ninja i18n – 🛎️ Translations need to be updated

Project /project.inlang

lint rule new reports level link
Missing translation 351 warning contribute (via Fink 🐦)

github-actions[bot] avatar Aug 27 '24 14:08 github-actions[bot]

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 73.22%. Comparing base (c122f0a) to head (b264dda). Report is 286 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #6090       +/-   ##
===========================================
- Coverage   86.74%   73.22%   -13.52%     
===========================================
  Files         217      238       +21     
  Lines        5032     6376     +1344     
  Branches      831     1023      +192     
===========================================
+ Hits         4365     4669      +304     
- Misses        561     1597     +1036     
- Partials      106      110        +4     
Flag Coverage Δ
appflowy_web_app 73.22% <ø> (-13.52%) :arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Aug 27 '24 14:08 codecov[bot]

Requires: https://github.com/AppFlowy-IO/AppFlowy-Collab/pull/273

This will enable the Card to display the sum of attachments across type options.

https://github.com/user-attachments/assets/62b40170-e034-4861-acae-cac59a3ec7f2

Xazin avatar Sep 03 '24 03:09 Xazin