Allow injecting available hosting plans into FunctionAppHostingPlanStep. Update DTS plans to include Flex Consumption
#4863
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
MicroFish91
changed the title
FunctionAppHostingPlanStep. Update DTS plans to include Flex ConsumptionFunctionAppHostingPlanStep. Update DTS plans to include Flex Consumption
motm32
approved these changes
Dec 9, 2025
MicroFish91
commented
Dec 9, 2025
| setPremiumPlanProperties(context); | ||
| } | ||
| } else if (context.useFlexConsumptionPlan) { | ||
| if (context.useFlexConsumptionPlan) { |
Contributor
Author
There was a problem hiding this comment.
I think there could be a better way to handle this now but I don't want to explore that in this PR. Basically, I think it would be better if we removed this and relied on the injected available plans to only show flex. Then we can auto-select if the pick was just the one pick. Prompt logic would then handle setting the related plan properties.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I added flex consumption to the list of available DTS plans. This mostly affects the Advanced Create flow.
I also changed the behavior of
FunctionAppHostingPlanStepa bit. I like the idea of injecting the available plans in as a dependency as it helps uncouple a lot of the more command-specific logic and helps with separation of concerns. This gives each command a chance to customize its own availability logic without having to worry about how it's going to affect other commands since this step can be shared anywhere.