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

feat(vscode): Introduce onboarding experience #3581

Merged
merged 31 commits into from
Nov 3, 2023
Merged

Conversation

ccastrotrejo
Copy link
Contributor

@ccastrotrejo ccastrotrejo commented Oct 31, 2023

Main Code Changes

  • Add extension settings for the auto download, auto azurite start, auto design time api start and dependencies path
  • Add logic to download binaries (.NET, NodeJS, Azure Functions Core Tools).
  • Add logic to execute binaries and update tasks.json

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

  • Overall introduces a new way users onboard with the extension

What is the current behavior? (You can also link to an open issue here)

  • Users need to download manually .NET, Azure Functions Core Tools, Azurite and NodeJS
  • Design time API triggers every time the user do right click in the workflow.json, sometimes this can last ~ 1 minute until the designer gets render
  • Azurite needs to be triggered every time users want to debug a logic app. Most of the users don't know this is a required step, which confuses the overall extension experience

What is the new behavior (if this is a feature change)?

  • Introduces option to users to auto download and validate dependencies binaries or keep using their system wide installations
  • Design time API now triggers automatically whenever the user opens a workspace with a logic app project, this helps to reduce the waiting time for designer no render making a smoother experiences
  • Azurite now triggers automatically whenever the user opens a workspace with a logic app project. This will help reduce the uncertainty that users have whenever they want to debug logic apps, removing the manual azurite step.

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

  • No manually changes on the applications by the users. If users opt in for the dependencies binaries, tasks.json will be auto updated

ynauls and others added 20 commits July 25, 2023 11:31
* Add first workspace changes

* Add validation for workspace and project

* +DesignTime settings

* +telemetry

* +PromptStartDesignTimeOption

* +Telemetry

* PR Feedback

* fix missing

---------

Co-authored-by: ccastrotrejo <[email protected]>
* Cherry Picked: binaryInstallation

* overwrite existing tasks.json

* Change wording

---------

Co-authored-by: Alan Kai (from Dev Box) <[email protected]>
* async Timeout for Dependencies

* fix timeout, fix async/await, add csharp setting

* Rename to timeOutErrorOperation

---------

Co-authored-by: Alan Kai (from Dev Box) <[email protected]>
* Add prompt to opt-in and opt-out for binaries installation

* Separate onboarding to different file

* Update onboarding ids

* Add default to binaries and update onboarding for binaries

* Add back functionality

* Separate functions according to binaries opt-in/out

* Add default value when get global setting

* Update dotnet command when there is no use of dotnet dependencies
* Add prompt to opt-in and opt-out for binaries installation

* Separate onboarding to different file

* Update onboarding ids

* Add default to binaries and update onboarding for binaries

* Add back functionality

* Separate functions according to binaries opt-in/out

* Add default value when get global setting

* Update dotnet command when there is no use of dotnet dependencies

* Update to default path when opt out

* Update strings to be able to review

* Update strings

* Add final point
* Update tasks.json according to binaries

* Add warning message for install binaries command
@ccastrotrejo ccastrotrejo merged commit f28c4fa into main Nov 3, 2023
11 checks passed
@ccastrotrejo ccastrotrejo deleted the devex/onboarding branch November 3, 2023 17:54
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.

4 participants