Skip to content

Conversation

@s-b-repo
Copy link

@s-b-repo s-b-repo commented Jan 3, 2026

fixes:

1.Added _currentPath state variable to safely store the path for display
2.Added null/empty check before setting Directory.current in initState
3.Added Directory(pref).existsSync() check to verify path exists
4.Use _currentPath in the button text instead of Directory.current.path
5.Save folder directly instead of Directory.current.path after setting****

Refactor loadOperatingSystems and getIcons functions for better error handling and code clarity. 

fixes:

    1.Wrapped getIcons() in try-catch to handle AssetManifest.json load failures gracefully
    2.Added await to getIcons() call for proper async error handling
    3.Added try-catch around PackageInfo.fromPlatform() (can also fail with sudo)
    4.Uses debugPrint for warnings instead of crashing
@s-b-repo
Copy link
Author

s-b-repo commented Jan 3, 2026

fixes:

1.Wrapped getIcons() in try-catch to handle AssetManifest.json load failures gracefully
2.Added await to getIcons() call for proper async error handling
3.Added try-catch around PackageInfo.fromPlatform() (can also fail with sudo)
4.Uses debugPrint for warnings instead of crashing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant