Node function fix (#212)

* Staging URLs

* Update remoteServiceUrlHelper.ts

* Review comments

* Review comments

* All webapps to remote for github repos

* function node fix

* function node fix

* fixed npm fail

* Adding comment

Author: tiwarishub

package.json

@@ -135,7 +135,7 @@
         "tslint-microsoft-contrib": "^6.2.0",
         "typescript": "3.3.1",
         "typescript-tslint-plugin": "^0.5.5",
-        "vscode": "1.1.33",
+        "vscode": "1.1.37",
         "vscode-azureextensiondev": "^0.3.1",
         "vscode-test": "^1.3.0"
     },

src/configure/configure.ts

@@ -116,8 +116,10 @@ class Orchestrator {
     }
 
     private doesLanguageAndTargetSupportRemoteProvisioning(): boolean {
+        // This check is to enable for all remote repository webapps and aks flows to use remote provisioning service.
+        // Both template type as remote and remote url check is required because remote provisioning is only applicable for remote templates and there are scenario where template selected is remote but repo is not remote (in cases where resource is already selected)
         return extensionVariables.remoteConfigurerEnabled === true && this.inputs.sourceRepository.repositoryProvider === RepositoryProvider.Github &&
-            (this.inputs.targetResource.resource.type === TargetResourceType.AKS || this.inputs.targetResource.resource.type === TargetResourceType.WebApp) && !!this.inputs.sourceRepository.remoteUrl;
+            (this.inputs.targetResource.resource.type === TargetResourceType.AKS || this.inputs.targetResource.resource.type === TargetResourceType.WebApp) && !!this.inputs.sourceRepository.remoteUrl && this.inputs.pipelineConfiguration.template.templateType === TemplateType.REMOTE;
     }
 
     private async getAzureResource(targetType: TargetResourceType) {