Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Version-agnostic App component #23699

Merged
merged 1 commit into from
Nov 1, 2021
Merged

Conversation

guardrex
Copy link
Collaborator

@guardrex guardrex commented Nov 1, 2021

Fixes #23696

Thanks @johndhunter! 🚀 ... This will be the best way to go for now. I'll revisit when I reach the security node on UE passes in 2022 and perhaps make it so that versioned code is loaded. The problem with doing that now is that:

  • It's a change that will require similar work in other areas. I might even decide in the end to use this "see the ref source" approach for everything. The framework churn is challenging to keep up with, and that's a guaranteed winner for churn as long as the dev selects the right release in the PU repo (per the INCLUDE here).
  • Or, we might be dropping this coverage entirely in 2022 in favor of the Azure docs coverage. If so, then we don't want to spend a lot of time making changes to this at this time. That's a waste of ⏱️ ... thus a waste of 💰 ... if it comes to pass that all of this gets dropped here.

@guardrex guardrex merged commit d46f3a2 into main Nov 1, 2021
@guardrex guardrex deleted the guardrex/blazor-auth-app-component-60 branch November 1, 2021 13:50
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.

Missing code which is generated from project template
1 participant