[Bug]: Deploy action does not recognize Bank Deposits Extension

[StefanMaron]

AL-Go version

6.4

Describe the issue

For some reason the deploy step tries to deploy the bank deposits app since it thinks its missing. But I have downloaded symbols with vs code from the very same environment just moments before.

Expected behavior

Should not try to deploy existing apps, and also not try to deploy symbols only .app files

Steps to reproduce

Not sure how.

Additional context (logs, screenshots, etc.)

No response

[freddydk]

Could you include or email me the full log in .txt format - it is hard to see anything from a screenshot
Thanks

[StefanMaron]

@freddydk Mail sent

[StefanMaron]

@freddydk any updates on this one? Its blocking my client right now.

[freddydk]

Sry - you say that it doesn't recognize Bank Deposits Extension - but the error you get is:

Deploying to DEV failed. Response status code does not indicate success: 401 (Unauthorized). Response status code does not indicate success: 401 (Unauthorized).

That looks like a different problem????

It looks like it installs Bank Deposits Extension from AppSource and then runs into an error like your S2S user doesn't have the right permissions or like?

[freddydk]

BTW - I think the logic on Deployment here was updated after v6.4 - could you also make sure that you use the latest version?

[StefanMaron]

Same issue on v7.0

But it should never try to install that app from appsource because it is installed since I can download the symbols for that app from the exact same environment.

Maybe its two separate issues. But the for me important one, that it should not try to install.
I never had/have issues with installation of my apps so far using those credentials, so permission wise it should be fine.

[freddydk]

Yeah, issue #1649 seems to be the exact same issue with a different app - I hop @spetersenms can look into this - else I will try to repro when I get back from the US.

[spetersenms]

I can't repro the issue, but looking at the error from this and the related issue, it definitely seems related to the exclude apps. I have a potential fix ready, but will need to test further Monday before deploying it.

[StefanMaron]

@spetersenms If I should test your fix on my pipeline, just let me know

[freddydk]

@StefanMaron - feel free to try out freddydk/AL-GO@issue1540
(Run update AL-Go System files with this string - you don't need https://github.com/ in front).

You can always switch back if this doesn't work.

[StefanMaron]

@freddydk it works now on your version.