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

Update the doc for alert status reason info and rename file to say Status Reason instead of Resolution Reason... #812

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ruchitpanw
Copy link

…-cloud-alert-resolution-reasons.adoc to prisma-cloud-alert-status-reasons.adoc

  • Updated the doc for correct alert status reason info as the current doc was mentioning the reason details for only resolved alerts BUT some of the reasons are also applicable to OPEN alerts
  • Renamed the file prisma-cloud-alert-resolution-reasons.adoc to prisma-cloud-alert-status-reasons.adoc to correct naming convention

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):

Fix #

Test URLs:

…-cloud-alert-resolution-reasons.adoc to prisma-cloud-alert-status-reasons.adoc

- Updated the doc for correct alert status reason info as the current doc was mentioning the reason details for only resolved alerts BUT some of the reasons are also applicable to OPEN alerts
- Renamed the file prisma-cloud-alert-resolution-reasons.adoc to prisma-cloud-alert-status-reasons.adoc to correct naming convention
@arane26 arane26 mentioned this pull request Sep 12, 2024
@arane26
Copy link
Collaborator

arane26 commented Sep 16, 2024

@ruchitpanw - this branch had some conflicts with our main branch. I created a new PR 898 and the suggested updates are now live.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants