AppFlowy icon indicating copy to clipboard operation
AppFlowy copied to clipboard

feat: cover in database row

Open Xazin opened this issue 1 year ago • 5 comments

Feature Preview

Update:

https://github.com/user-attachments/assets/3634988b-de0c-4f15-a26c-ec5c4c647f4b

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.
  • [ ] 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 Sep 08 '24 22:09 Xazin

🥷 Ninja i18n – 🛎️ Translations need to be updated

Project /project.inlang

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

github-actions[bot] avatar Sep 08 '24 22:09 github-actions[bot]

Codecov Report

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

Project coverage is 74.92%. Comparing base (c122f0a) to head (b8bc0f0). Report is 351 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #6238       +/-   ##
===========================================
- Coverage   86.74%   74.92%   -11.83%     
===========================================
  Files         217      239       +22     
  Lines        5032     6492     +1460     
  Branches      831     1045      +214     
===========================================
+ Hits         4365     4864      +499     
- Misses        561     1576     +1015     
+ Partials      106       52       -54     
Flag Coverage Δ
appflowy_web_app 74.92% <ø> (-11.83%) :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 Sep 08 '24 22:09 codecov[bot]

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

Xazin avatar Sep 08 '24 22:09 Xazin

Need to add tests, after that it should be ready.

Xazin avatar Sep 10 '24 16:09 Xazin

Turned into a draft until I resolve a final issue with notifying - will push the fix along with some tests tomorrow. :+1:

Xazin avatar Sep 12 '24 23:09 Xazin