AppFlowy icon indicating copy to clipboard operation
AppFlowy copied to clipboard

feat: files media improvements

Open Xazin opened this issue 1 year ago β€’ 4 comments

PR Checklist

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

Xazin avatar Oct 28 '24 00:10 Xazin

πŸ₯· Ninja i18n – πŸ›ŽοΈ Translations need to be updated

Project /project.inlang

lint rule new reports level link
Missing translation 189 warning contribute (via Fink 🐦)
Message without source 2 warning contribute (via Fink 🐦)

github-actions[bot] avatar Oct 28 '24 00:10 github-actions[bot]

Codecov Report

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

Project coverage is 65.02%. Comparing base (7ecf213) to head (e111257). Report is 215 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6632      +/-   ##
==========================================
- Coverage   74.41%   65.02%   -9.39%     
==========================================
  Files         245      273      +28     
  Lines        6566    10471    +3905     
  Branches     1057     1434     +377     
==========================================
+ Hits         4886     6809    +1923     
- Misses       1628     3494    +1866     
- Partials       52      168     +116     
Flag Coverage Ξ”
appflowy_web_app 65.02% <ΓΈ> (-9.39%) :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 Nov 01 '24 05:11 codecov[bot]

Do we have tests for file media?

appflowy avatar Nov 01 '24 05:11 appflowy

Do we have tests for file media?

We have integration tests on the frontend.

Xazin avatar Nov 01 '24 05:11 Xazin