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

[ALM Accelerator - BUG]: Upgrade solution tick box getting unticked #5882

Closed
ryanhodgson1 opened this issue Jun 18, 2023 · 5 comments · Fixed by #5775
Closed

[ALM Accelerator - BUG]: Upgrade solution tick box getting unticked #5882

ryanhodgson1 opened this issue Jun 18, 2023 · 5 comments · Fixed by #5775
Assignees
Labels
alm-accelerator ALM Accelerator Components and Apps bug Something isn't working

Comments

@ryanhodgson1
Copy link

Describe the issue

Seems like in the new release there is an issue with the upgrade solution tick box not getting retained. I can tick the box, save the settings and then re-open the settings but the box is unticked again:

image

Expected Behavior

For the tick to be retained.

What component are you experiencing the issue with?

ALM Accelerator Canvas App

What solution version are you using?

1.0.20230606.1

Steps To Reproduce

No response

Anything else?

No response

@ryanhodgson1 ryanhodgson1 added alm-accelerator ALM Accelerator Components and Apps bug Something isn't working labels Jun 18, 2023
@mikefactorial mikefactorial self-assigned this Jun 26, 2023
@mikefactorial
Copy link
Collaborator

Thanks again @RyanHodgson! Looks like the default value on the tickbox got changed in a previous release. This will be fixed in the July release next week.

CoEStarterKitBot pushed a commit that referenced this issue Jun 26, 2023
@ryanhodgson1
Copy link
Author

thanks @mikefactorial

Are you able to guide me to where i can change this until then please?

@mikefactorial
Copy link
Collaborator

@ryanhodgson1 you'll have to edit the canvas app deployment settings screen. The default value of the Upgrade Solution checkbox should be

If(solutionupgradetag="true", true, false)

May already be aware but if you make the change it will create an unmanaged layer and future releases that update the app might not apply until you remove the unmanaged layer.

image

@ryanhodgson1
Copy link
Author

Thanks @mikefactorial i have updated that now and will remove the layer before upgrading to the next version.

@CoEStarterKitBot
Copy link
Collaborator

@ryanhodgson1 This has been fixed in the latest release. Please install the latest version of the toolkit following the instructions for installing updates. Note that if you do not remove the unmanaged layers as described there you will not receive updates from us.

ChristopheHare pushed a commit to ChristopheHare/coe-starter-kit that referenced this issue Sep 7, 2023
* Fixes microsoft#5370

* Fixes microsoft#5794

* Fixes microsoft#5891

* Fixes microsoft#5882

* Removing dependencies

* Pushing Latest solution

---------

Co-authored-by: Michael Peltier <Michaelp@powercattools.onmicrosoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
alm-accelerator ALM Accelerator Components and Apps bug Something isn't working
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants