feat!: Rewrite major parts of the file browser #1873

MoritzWeber0 · 2024-10-02T21:09:18Z

Breaking API change: filename was renamed to path in /api/v1/sessions/{session_id}/files/download to make it consistent with the the list files endpoint.

The UI got an overhaul - margins are fixed and more icons for different cases were added (added files, modified files, expanded folder)
The workspace is now expanded per default
Download of single files (not directories) was added, resolves Download single files in file browser #1286
Replaced brute force to find a path with path traversal, which is a lot more efficient
Use the generated OpenAPI client, fix API docs to use ZIP as response
Migrate to TailwindCSS
Remove custom "Overwrite file" dialog, replace with ConfirmationDialog.
Remove NestedTreeControl, resolves Remove usage of NestedTreeControl #1732
Separate handling for files which replace existing files
Stream downloading directory directory without loading to memory
Fix a bug that files with special characters couldn't be downloaded (archive had a size of 0KB)

Resolves #118 except upload limit (will be kept to reduce load on the backend) + different file icons for different extensions (not planned).

codecov · 2024-10-02T21:11:40Z

Codecov Report

Attention: Patch coverage is 88.88889% with 1 line in your changes missing coverage. Please review.

Project coverage is 84.06%. Comparing base (8601089) to head (35c1aa1).
Report is 6 commits behind head on main.

Files with missing lines	Patch %	Lines
backend/capellacollab/sessions/files/routes.py	75.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1873      +/-   ##
==========================================
- Coverage   84.44%   84.06%   -0.38%     
==========================================
  Files         197      197              
  Lines        6549     6716     +167     
  Branches      729      768      +39     
==========================================
+ Hits         5530     5646     +116     
- Misses        859      904      +45     
- Partials      160      166       +6

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

github-actions · 2024-10-02T21:12:50Z

A Storybook preview is available for commit 37fc7a3.
➡️ View Storybook
➡️ View Chromatic build

...user-sessions-wrapper/active-sessions/file-browser-dialog/file-browser-dialog.component.html

Breaking API change: `filename` was renamed to `path` in `/api/v1/sessions/{session_id}/files/download` to make it consistent with the the list files endpoint. - The UI got an overhaul - margins are fixed and more icons for different cases were added (added files, modified files, expanded folder) - The workspace is now expanded per default - Download of single files (not directories) was added - Replaced brute force to find a path with path traversal, which is a lot more efficient - Use the generated OpenAPI client, fix API docs to use ZIP as response - Migrate to TailwindCSS - Remove custom "Overwrite file" dialog, replace with ConfirmationDialog. - Remove `NestedTreeControl` - Separate handling for files which replace existing files - Stream downloading directory directory without loading to memory - Fix a bug that files with special characters couldn't be downloaded (archive had a size of 0KB)