Xia Zhao

This issue has been fixed by https://github.com/aws/serverless-application-model/pull/2408. Closing this issue now. Feel free to reopen it if there are any questions.

Tested the feature and it's working as expect. Closing this issue. Feel free to re-open the issue if you have any other questions.

@MLaunch Thanks for posting this issue. It looks like a feature request to me. Could you please use the feature request template and add some details to each section? Thanks!

Hello @kshyun28 , For new policy template PR, it has to go through security review which will take some time. We won't be able to merge the PR now.

As @GavinZZ mentioned, `addManagedPolicy` is not implemented because it's in the abstract class. In the extended `Group` class, the function is implemented. https://github.com/aws/aws-cdk/blob/v2.80.0/packages/aws-cdk-lib/aws-iam/lib/group.ts#L213 Close the issue for now. Feel free...

Can't reproduce this issue. Looking at the internal tracking ticket, this happens when `encryptionKey: dummyKey` creates the policy with same action/condition as `crossAccountAccessPolicy` which will cause conflicts. When I run...

Starting to look into this PR. Because I'm not familiar with this L2, could you please add some details to help me understand the change? - Reason of changes -...

I agree separating `managed/unmanaged` seems unnecessary here. `unmanaged` should be able handle all cases. However there might be some edge cases using `managed` path that we're not aware of. Removing...

@Mergify update

@shadywallas Thanks for asking. We already added it to our queue for internal review. Will post here if we have any updates.