Skip to content

chore: Apply split - #1

Merged
baillyjamy merged 491 commits into
mainfrom
apply-split
Aug 4, 2026
Merged

chore: Apply split#1
baillyjamy merged 491 commits into
mainfrom
apply-split

Conversation

@baillyjamy

Copy link
Copy Markdown
Contributor

No description provided.

tevincent and others added 30 commits August 3, 2026 14:46
# Conflicts:
#	app/build.gradle.kts
#	gradle/libs.versions.toml

wip

wip

# Conflicts:
#	multiplatform-lib/src/commonMain/kotlin/AuthenticatorFacade.kt
#	multiplatform-lib/src/commonMain/kotlin/internal/AuthenticatorFacadeImpl.kt
#	multiplatform-lib/src/commonMain/kotlin/internal/managers/MigrationManager.kt

wip

# Conflicts:
#	multiplatform-lib/src/commonMain/kotlin/internal/AuthenticatorFacadeImpl.kt

# Conflicts:
#	app/build.gradle.kts
# Conflicts:
#	multiplatform-lib/src/commonMain/kotlin/AuthenticatorFacade.kt
#	multiplatform-lib/src/commonMain/kotlin/DummyAuthenticatorFacade.kt
#	multiplatform-lib/src/commonMain/kotlin/internal/AuthenticatorFacadeImpl.kt
tevincent and others added 26 commits August 3, 2026 14:47
Fixed by setting up AuthConfiguration for refresh to work
and re-fetching the token from the DB on retry,
so we can get the refreshed one.
If the completeMigration API call failed,
the user would end up locked for the given account.

It turns out the completeMigration API call is not mandatory
to fully migrate the app, and it might be replaced by the
detection of passkey use, so we're now doing a fire and forget,
until we can completely remove it.
Before, it was hidden by our app bar on Android,
because we had incorrect padding.
It was caused by the accounts flow not yet in sync with the db contents.
Now, instead of using the accountEntities Flow, we use the accounts one,
which has been updated to reflect that we're about to start migration.

This commit also adds some clarifying comments to explain the desired behavior.
This will allow it to be used across multiple "targets" in the iOS project.
@baillyjamy baillyjamy changed the title Apply split chore: Apply split Aug 3, 2026
@baillyjamy
baillyjamy merged commit d7ec3d9 into main Aug 4, 2026
@baillyjamy
baillyjamy deleted the apply-split branch August 4, 2026 08:42
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.

5 participants