Skip to content

fix: merge appState in Auth Guard#543

Open
frederikprijck wants to merge 1 commit intomainfrom
fix/434
Open

fix: merge appState in Auth Guard#543
frederikprijck wants to merge 1 commit intomainfrom
fix/434

Conversation

@frederikprijck
Copy link
Member

Changes

When a user would provide appState on redirectLoginOptions when creating the Auth Guard, we would lose any appState.target value, due to the fact that this was not being accounted for.

References

Closes #434

Testing

  • This change adds unit test coverage
  • This change adds integration test coverage
  • This change has been tested on the latest version of the platform/language

Checklist

@frederikprijck frederikprijck requested a review from a team as a code owner February 13, 2026 11:25
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.

Merge AuthGuardOptions.redirectLoginOptions.appState in AuthGuard

1 participant