Skip to content

feat(startup): add migration preparation screen PR 2 - #5159

Open
MohamadJaara wants to merge 1 commit into
mo/async-db-migration/01-foundationfrom
mo/async-db-migration/02-migration-screen
Open

feat(startup): add migration preparation screen PR 2#5159
MohamadJaara wants to merge 1 commit into
mo/async-db-migration/01-foundationfrom
mo/async-db-migration/02-migration-screen

Conversation

@MohamadJaara

Copy link
Copy Markdown
Member

Goal

Show long-running user database preparation as a continuation of the splash experience, with clear status and recovery actions.

Changes

  • reuse the splash logo and visual layout in a static migration screen
  • map every preparation state and failure to user-facing copy and actions
  • avoid flashing the screen for fast database opens
  • add previews for every supported state and string variant

Stack

  • Epic branch: mo/epic/async-db-migration
  • Previous: PR 1
  • PR 2 of 7

Merge the stack bottom-up into the epic branch.

Comment on lines +68 to +69
.background(colorResource(R.color.background))
.padding(horizontal = 32.dp),

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

All dimensions and colors are hardcoded in this file, can we use colorsScheme().background and dimensions().spacing32x instead?

@MohamadJaara
MohamadJaara force-pushed the mo/async-db-migration/02-migration-screen branch 2 times, most recently from 5f9170a to 4e62773 Compare August 18, 2026 12:45
@github-actions

github-actions Bot commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Test Results

381 tests   381 ✅  24s ⏱️
 40 suites    0 💤
 40 files      0 ❌

Results for commit 162c722.

♻️ This comment has been updated with latest results.

@MohamadJaara
MohamadJaara force-pushed the mo/async-db-migration/02-migration-screen branch from 4e62773 to 162c722 Compare August 19, 2026 11:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants