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.

Sign up for GitHub

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

fix: fixes for downloading #23

Merged
merged 9 commits into from
Jul 19, 2024
Merged

fix: fixes for downloading #23

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
25c821b
fix: need to delete downloaded and waiting/inProgress state when tap …
rnr Jul 5, 2024
f8b4082
chore: delete unused print
rnr Jul 5, 2024
ae5b21c
chore: added Destination for downloadRequest
rnr Jul 8, 2024
53ec9fd
Merge branch 'develop' into small-fix-for-downloading-cancelling
rnr Jul 9, 2024
8ecb4f6
Merge branch 'develop' into small-fix-for-downloading-cancelling
rnr Jul 9, 2024
ef08ff2
Merge branch '2U/develop' into small-fix-for-downloading-cancelling
rnr Jul 17, 2024
f886ce3
chore: fixed doubled code after merge
rnr Jul 17, 2024
2b0fe97
fix: downloads count after cancellation
forgotvas Jul 17, 2024
43f0325
chore: fixed lint warning
rnr Jul 18, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion Core/Core/Network/DownloadManager.swift
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -268,7 +268,8 @@ public class DownloadManager: DownloadManagerProtocol {
public func deleteFile(blocks: [CourseBlock]) async {
for block in blocks {
do {
if let fileURL = fileUrl(for: block.id) {
if let fileURL = fileUrl(for: block.id),
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Getting Trailing Whitespace Violation: warning here.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

FileManager.default.fileExists(atPath: fileURL.path) {
try FileManager.default.removeItem(at: fileURL)
}
try await persistence.deleteDownloadDataTask(id: block.id)
Expand Down
Loading