Skip to content

feat: restore unfinished document drafts - #28

Merged
lorenzocamilli merged 3 commits into
FOSScanner:devfrom
VincenzoImp:fix/document-draft-recovery
Sep 3, 2026
Merged

feat: restore unfinished document drafts#28
lorenzocamilli merged 3 commits into
FOSScanner:devfrom
VincenzoImp:fix/document-draft-recovery

Conversation

@VincenzoImp

Copy link
Copy Markdown
Contributor

Layer 2 of the follow-up stack announced in #25, rebased onto the updated dev.

Summary

  • persist unfinished document drafts atomically in the app-private, OS-managed cache on native platforms
  • restore a valid current or backup draft after process death, and serialize restore/save operations
  • exclude app data from Android OS backups (android:allowBackup="false") so drafts are never copied off-device
  • guard navigation while processing is in flight
  • add confirmations or Undo for destructive actions, including an explicit clear/keep choice after sharing
  • keep web persistence an intentional no-op (in-progress pages stay memory-only, as before)
  • document the retention behavior in README, SECURITY, the F-Droid description, and the feature-request template

Verification

  • formatter and flutter analyze clean
  • draft, recovery, navigation, destructive-action, and corruption tests pass
  • complete stack: 138 tests pass
  • the rebase onto current dev was conflict-free, and the resulting tree is identical to the verified one apart from the 1.3.0 release bump

Remaining follow-ups

Each will be rebased onto dev and opened here after this layer merges:

  1. Bound and serialize image processing
  2. Improve scanner accessibility and platform support

@VincenzoImp VincenzoImp closed this Sep 2, 2026
@VincenzoImp VincenzoImp reopened this Sep 2, 2026
@lorenzocamilli
lorenzocamilli merged commit 9fce1b9 into FOSScanner:dev Sep 3, 2026
11 of 12 checks passed
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.

2 participants