Skip to content

Add auth method to authentication state.#3463

Merged
sea-snake merged 1 commit intomainfrom
sea-snake/auth-method-in-state
Nov 4, 2025
Merged

Add auth method to authentication state.#3463
sea-snake merged 1 commit intomainfrom
sea-snake/auth-method-in-state

Conversation

@sea-snake
Copy link
Copy Markdown
Contributor

@sea-snake sea-snake commented Nov 4, 2025

Add auth method to authentication state, this will be used in the dashboard to indicate which access method is currently in use.

The last used identity store data could also be used for this purpose but isn't reliable since it's value could change by signing in on other tabs/windows due to it being tied to local storage.

Changes

  • Add authMethod property to authentication store state.
  • Update flows to pass auth method into authentication store.
  • Remove unused confirmationCode from authFlow.

Tests

Verified authentication flows still work as expected and correct auth method value is present when expected.

@sea-snake sea-snake requested a review from lmuntaner November 4, 2025 13:24
@sea-snake sea-snake enabled auto-merge November 4, 2025 13:34
@sea-snake sea-snake added this pull request to the merge queue Nov 4, 2025
Merged via the queue into main with commit a62f661 Nov 4, 2025
76 checks passed
@sea-snake sea-snake deleted the sea-snake/auth-method-in-state branch November 4, 2025 13:46
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