UX: Change command labels, remove entry point for Create new project...
, add Create Function App
from deploy
#4042
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.
This changes the Functions UI a bit to match what we discussed offline and insights from user studies.
Create New Project...
. ReplaceCreate Function...
withCreate function locally...
(Not sure what looks best in terms of casing)If you're unfamiliar,
Create Function...
handles the following scenarios:Added two entry points to
Create function locally...
. One on theFunctions
node indicated by the Function + button. Another as a context menu action. I'm debating moving it to the local project node.Mentioned renaming
Create Function...
but also renamedDeploy to Function App in Azure...
toDeploy function to Azure...
. It makes more intuitive sense because your local project is being deployed to Azure, and we have to prompt for which app is being deployed to.By that logic, I've kept
Deploy to Function App...
the same in theResources
view though.Added
Create New Project...
to workspace view if there's no project in the workspace. It actually does callazureFunctions.createNewProject
because we know that we need to start from scratch.createFunction
has to do some checks before it ultimately cancels and callscreateNewProject
so performance is slightly better to just call it directly.Removed
Create Function App...
option in the local workspace viewDeploy function to Azure...
which includes a+ Create new function app
option as a quick pick when we prompt for the appMerged
Create New Containerized Project...
intoCreate function locally...
and add it as a prompt at the beginning. It's annoying, but also a one-time cost to the user that can be easily entered through.