Skip to content

Conversation

@farook-edev
Copy link
Contributor

Added Dialog for when an offline (not connected to the internet) user attempts to download resources, the error only occurs in case a SocketException is caught, any other exceptions are still unhandled.

This fixes #948

image

@farook-edev farook-edev requested review from a team and anhappdev as code owners January 30, 2025 11:32
@github-actions
Copy link

github-actions bot commented Jan 30, 2025

MLCommons CLA bot All contributors have signed the MLCommons CLA ✍️ ✅

@farook-edev farook-edev mentioned this pull request Jan 30, 2025
@anhappdev
Copy link
Collaborator

@farook-edev We still encounter cases where a downloaded file becomes corrupted, primarily due to connection drops. In such instances, the app should automatically delete the corrupted file. If you prefer, we can create another pull request for this feature.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 4, 2025

@farook-edev farook-edev merged commit 399ee37 into master Feb 5, 2025
19 checks passed
@farook-edev farook-edev deleted the offline-dialog branch February 5, 2025 13:57
@github-actions github-actions bot locked and limited conversation to collaborators Feb 5, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

App should notify users that the device is offline when trying to download files

4 participants