-
Notifications
You must be signed in to change notification settings - Fork 0
116 reject big files #118
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
Open
broija
wants to merge
78
commits into
main
Choose a base branch
from
116_reject_big_files
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
116 reject big files #118
+5,313
−56
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Added download store for keeping track of downloaded files. - On click on "Stop hosting" : all files are deleted. - Added gradient outline for "Start hosting" button on hover. - Added dummy download tools (test only).
[FEAT] About popup (#33_about_popup)
21 start stop buttons
40 fix macos build ctd
[FEAT] Add log window (#28_log_window)
[BUILD] Fix win build style (#41_fix_win_build_style)
20250821 dawnlaod
* [FEAT] Cleanup screen (#52_cleanup_screen) - Now "Stop hosting" triggers cleanup screen display. - Download store service turned into a class. * [FEAT] Tests, fixes and modifications for "demo" version (#75_tests_fixes_for_demo) - IPC : - "fetch-climate-data" renamed to "rescue-api:call" and provide only the route and not the complete URL. - "download-file" accepts a default name prefix. - HTTP file download : - the app now attempts to guess the filename from Content-Disposition header or URL. - file size is calculated after download. Size guessing is used by default. - asset.name is ignored, because irrelevant most of the times actually. - Moved window positions at start. - Rescue API : - Asset model modified. - rescuer_id is hardcoded to 1, a value existing in DB. - Added some debug messages. - Configuration : - API URL : Production URL is now hardcoded. Dev one is configurable. - Modified .env.example for switching between mock and local API. * [FIX] Fix main window on start for Windows platform (#51_fix_windows_main_window) * [FIX] (#75_tests_fixes_for_demo) - Fixed node_id to be consistent with rescuer_id - Added log level filter for production * [FIX] Fix main window path displayed on Windows (#51_fix_windows_main_window) * [FIX] Fixed Windows free space calculation (#51_fix_windows_main_window) * [BUILD] Fix URL config for production (#75_tests_fixes_for_demo)
54 tune styles before demo
- Storage limit saved. - Calculating remaining free space after each download. - Added FreeSpaceExhausted component.
- Added user config. - Added IPC message to check free space. - Storage selection : reduce custom selection button sizes. - FreeSpaceExhausted component : - Added buttons. - Turned into success component. - DispatchRequestPayload : node_id is now a number. - Dispatcher : free space sent is now remaining allocated space.
- Now files are downloaded until allocated space limit is reached or nothing has been downloaded. - node_id : temporarily switched back to string for API compatibility. - Failed donwloads are now distinguished from others, but no failure handling yet.
- UserConfig : create ~/.shelter-config.json when necessary. - Simplified registerSystemInfo for bandwith limitation.
Fixed free space check returns in nextDownload and downloadAssets.
- Added banwith limitation component. - Modified window expansion logic. - UserConfig : - added cache. - values can be undefined. - Added BandwidthLimiterService (WIP) - downloadFile function is now nested into DownloadService class. -
17 enforce storage limit
65 download cycle
* [FEAT] API error handling (#95_handle_api_errors) - Added RescueApiService. - Added API error and timeout handling. * [DOC] Added some comments to handleRetry (rescue-api-service) (#95_handle_api_errors)
- Asset id is always added before file name. - Files bigger than 400MB are rejected and reported to API. - Defective links are reported to API. - Exchanged sizes with API are now expressed in bytes.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.