microsoft / navcontainerhelper

Official Microsoft repository for BcContainerHelper, a PowerShell module, which makes it easier to work with Business Central Containers on Docker.
MIT License
385 stars 247 forks source link

Run-AlPipeline 'appFolders' because it is an empty array #1377

Closed dfredborg closed 4 years ago

dfredborg commented 4 years ago

I am getting an error: image

when I try to run the following script: image

And if i try to say okay then I will just add an extra app folder so it looks like this:

-appFolders @("app,app") `

I get the following error:

image

The structure of my app is as follows:

image

What am I doing wrong 🤔

freddydk commented 4 years ago

The function accepts either an array or a string if you had a folder (next to the app folder) called app2 it would be

-appFolders @("app","app2) or -appFolders "app,app2"

@("app","app2) is an array with to elements @("app,app2") is an array with one element (which is a string) "app,app2" is a string

dfredborg commented 4 years ago

I am still getting an error

image

I only have one App folder so I do not really need an array

freddydk commented 4 years ago

Cannot repro this - tried to run exactly the same script as above (in 1.0.9)

Tries this:

Run-AlPipeline -pipelineName "xx" -licenseFile "C:\temp\license.flf" -baseFolder "C:\Users\freddyk\Documents\GitHub\BusinessCentralApps\BingMaps - Copy" -appFolders "app" -enablePerTenantExtensionCop -enableUICop

(thought it could be something with the space in the basefolder" - but it works.

freddydk commented 4 years ago

What is the content of your app folder above? Where is your app.json file?

dfredborg commented 4 years ago

I have tried to run it on two different projects but with the same outcome:

image

and the simple one:

image

The funny thing is that it worked, until it did not, so I do not know if there has been a docker update or windows update.

freddydk commented 4 years ago

This is not due to docker or windows updates. Shoot me an email to freddyk at microsoft dot com and lets setup a 15 minute teams call - I would like to see this (other people might hit the same).

dfredborg commented 4 years ago

Unfortunately the new commit did not solve my problem ☹

image

freddydk commented 4 years ago

The fix is not included in any release yet (build/release process takes 2+ hours)

/Freddy

freddydk commented 4 years ago

Shipped in 1.0.10-preview224

dfredborg commented 4 years ago

I works thanks for the help 😊

freddydk commented 4 years ago

Fix shipped in 1.0.10