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

properly handle missing managed policies #319

Merged
merged 2 commits into from
Oct 5, 2023

Conversation

gruebel
Copy link
Collaborator

@gruebel gruebel commented Oct 4, 2023

What does this PR do?

  • properly handling the case of a managed policy being not there anymore (ex. when it was renamed)

Fixes #318

What gif best describes this PR or how it makes you feel?

Completion checklist

  • Additions and changes have unit tests
  • The pull request has been appropriately labeled using the provided PR labels
  • GitHub actions automation is passing (make test, make lint, make security-test, make test-js)
  • If the UI contents or JavaScript files have been modified, generate a new example report:
# Generate the updated Javascript bundle
make build-js

# Generate the example report
make generate-report

@gruebel gruebel added the bug Something isn't working label Oct 4, 2023
@gruebel gruebel merged commit 8acac6a into salesforce:master Oct 5, 2023
@gruebel gruebel deleted the handle-missing-policy branch October 5, 2023 12:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working cla:signed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ARN Not found
1 participant