Skip to content

Conversation

guardrex
Copy link
Collaborator

@guardrex guardrex commented Aug 25, 2025

Fixes #35958

Thanks @jbw716! 🚀

Mackinnon ... Updated per your guidance.

@guardrex guardrex self-assigned this Aug 25, 2025
@guardrex guardrex requested a review from Copilot August 25, 2025 15:31
Copilot

This comment was marked as outdated.

@guardrex
Copy link
Collaborator Author

One more thing Mackinnon ....

I don't like the way that they named the snapshot options "Options" because it fouls up removing the fully-qualified naming for Options.DefaultName, so I'm changing the suggested code on the PR to make the snapshot options "RemoteOptions."

Updated the injection of options and modified the redirect logic in the OnInitialized method.
@guardrex guardrex requested a review from MackinnonBuck August 25, 2025 15:38
Updated instructions for customizing the login path in the RedirectToLogin component, including code examples for using IOptionsSnapshot.
Updated the text to clarify the default login path format.
@guardrex guardrex mentioned this pull request Aug 2, 2025
86 tasks
@guardrex
Copy link
Collaborator Author

guardrex commented Sep 3, 2025

@MackinnonBuck ... Free for a fast 🏃‍♂️ review? Under the new doc process guidelines (other repos are slow, so this one should be slow too!), I need every bleed'in little change reviewed 😠.

Copy link
Member

@MackinnonBuck MackinnonBuck left a comment

Choose a reason for hiding this comment

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

Approving with some minor suggestions. Looks good!

@guardrex guardrex merged commit 772e1c5 into main Sep 3, 2025
3 checks passed
@guardrex guardrex deleted the guardrex-patch-3 branch September 3, 2025 16:22
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.

Blazor MSAL RedirectToLogin.razor reference source out of date
2 participants