[CoE Starter Kit - BUG] Admin | add Maker to a group - success even if the owner doesn't have the sufficient privilege to add the member

[tocri]

Does this bug already exist in our backlog?

• I have checked and confirm this is a new bug.

Describe the issue

hi,

I forgot to add my admin coe user as owner of the AAD security group that i use for makers.
The flow status is successful even if the Add member to a group action fails.

I don't really know if it's a bug or if it's intentional.

For my part, I was surprised after a few days to see that there were no members in this group.

Expected Behavior

maybe make the flow fail so that we can see it in the COE admin command center?

What solution are you experiencing the issue with?

Core

What solution version are you using?

September

What app or flow are you having the issue with?

Admin [ Add Maker to a group

What method are you using to get inventory and telemetry?

None

Steps To Reproduce

No response

Anything else?

No response

AB#1414

[Jenefer-Monroe]

It was intentional in that we were catching the case where the user is already a member.
But I can make this better to fail for other things like this.
Thank you for posting!

[Jenefer-Monroe]

Fix is to only try to add if not already a member. This way if it fails it wont be for a reason we want to catch.

[CoEStarterKitBot]

@tocri 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.