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

Add offline dialog #954

Merged
merged 7 commits into from
Feb 5, 2025
Merged

Add offline dialog #954

merged 7 commits into from
Feb 5, 2025

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 anhappdev and a team as code owners January 30, 2025 11:32
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
flutter/lib/ui/settings/resources_screen.dart Outdated Show resolved Hide resolved
@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.

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
3 participants