Skip to content

revert: undo the accidental merge of #2459 and #2478 into master - #2487

Merged
russellwheatley merged 2 commits into
masterfrom
revert/version-10.0.0-beta05-accidental-merge
Sep 7, 2026
Merged

revert: undo the accidental merge of #2459 and #2478 into master#2487
russellwheatley merged 2 commits into
masterfrom
revert/version-10.0.0-beta05-accidental-merge

Conversation

@demolaf

@demolaf demolaf commented Sep 7, 2026

Copy link
Copy Markdown
Member

#2459 was merged into master unintentionally, and #2478 - retargeted from version-10.0.0-beta05 to master when that merge landed - went in two seconds later, stacked on top of it.

This reverts both in order (#2478, then #2459), putting master's tree back exactly on fa2dec53. Reverting #2459 on its own conflicts in 29 files, since #2478 builds on it.

Neither PR can be reopened - GitHub refuses once a PR is merged - so the work is open again as #2486, from the same version-10.0.0-beta05 branch, fast-forwarded to include #2478's commits.

@gemini-code-assist gemini-code-assist Bot left a comment

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.

Code Review

This pull request refactors the authentication flow to use Jetpack Navigation instead of the custom Navigation 3 implementation. It removes the dependency on Navigation 3 and Kotlin Serialization, simplifies the reauthentication flow by introducing an AuthState.ReauthenticationRequired state, and updates the MFA enrollment flow to include a recovery code step. Additionally, it removes several test tags and their associated resource ID exposure logic, opting for simpler testing approaches. No specific issues were identified in the review comments as there were no comments provided.

@russellwheatley
russellwheatley merged commit 9684270 into master Sep 7, 2026
18 checks passed
@russellwheatley
russellwheatley deleted the revert/version-10.0.0-beta05-accidental-merge branch September 7, 2026 12:11
@github-project-automation github-project-automation Bot moved this from Backlog to Done in studio-2394994192-60a69 Sep 7, 2026
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