Skip to content
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

refactor(files_app): Restructure Blocs #2215

Merged
merged 1 commit into from
Aug 16, 2024

Conversation

provokateurin
Copy link
Member

Simplifies the Bloc structure and improves the overall architecture of the Files app.

This is probably the right moment to add tests for everything, but that will be quite some effort.
Even without tests this can be reviewed and tested already.

@provokateurin provokateurin requested a review from Leptopoda June 29, 2024 15:31
@provokateurin provokateurin force-pushed the refactor/neon_framework/options-provider branch from ca142ff to 3b9f88c Compare July 9, 2024 12:24
@provokateurin provokateurin force-pushed the refactor/neon_files/restructure-blocs branch from dba77b8 to 6079009 Compare July 9, 2024 16:55
Base automatically changed from refactor/neon_framework/options-provider to main July 14, 2024 17:46
@provokateurin provokateurin force-pushed the refactor/neon_files/restructure-blocs branch from 6079009 to aa72fb1 Compare August 15, 2024 12:41
@provokateurin provokateurin changed the title refactor(neon_files): Restructure Blocs refactor(files_app): Restructure Blocs Aug 15, 2024
Copy link

codecov bot commented Aug 15, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 26.29%. Comparing base (955f599) to head (4080f84).
Report is 18 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2215   +/-   ##
=======================================
  Coverage   26.29%   26.29%           
=======================================
  Files         362      362           
  Lines      136468   136470    +2     
=======================================
+ Hits        35889    35891    +2     
  Misses     100579   100579           
Flag Coverage Δ *Carryforward flag
cookie_store 100.00% <ø> (ø) Carriedforward from 955f599
dashboard_app 96.05% <ø> (ø)
dynamite 31.08% <ø> (ø) Carriedforward from 955f599
dynamite_end_to_end_test 61.84% <ø> (ø) Carriedforward from 955f599
dynamite_runtime 85.40% <ø> (ø) Carriedforward from 955f599
neon_dashboard 96.05% <ø> (ø) Carriedforward from 955f599
neon_framework 45.83% <ø> (ø)
neon_http_client 93.28% <ø> (ø) Carriedforward from 955f599
neon_notifications 100.00% <ø> (ø) Carriedforward from 955f599
neon_talk 99.45% <ø> (ø) Carriedforward from 955f599
nextcloud 22.29% <ø> (ø) Carriedforward from 955f599
notifications_app 100.00% <ø> (ø)
sort_box 90.90% <ø> (ø) Carriedforward from 955f599
talk_app 99.36% <ø> (+<0.01%) ⬆️

*This pull request uses carry forward flags. Click here to find out more.

see 1 file with indirect coverage changes

@provokateurin provokateurin force-pushed the refactor/neon_files/restructure-blocs branch from aa72fb1 to 4080f84 Compare August 16, 2024 07:30
@provokateurin provokateurin marked this pull request as ready for review August 16, 2024 07:30
@provokateurin provokateurin merged commit 3dedeb1 into main Aug 16, 2024
8 checks passed
@provokateurin provokateurin deleted the refactor/neon_files/restructure-blocs branch August 16, 2024 07:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants