-
-
Notifications
You must be signed in to change notification settings - Fork 138
2.17 Headline changes and Alert details #623
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
base: main
Are you sure you want to change the base?
Conversation
0d03669 to
b1f5f34
Compare
|
Great job! No new security vulnerabilities introduced in this pull requestUse @Checkmarx to reach out to us for assistance. Just send a PR comment with Examples: |
| <a href="https://www.zaproxy.org/docs/alerts/">https://www.zaproxy.org/docs/alerts/</a> | ||
| </p> | ||
|
|
||
| <a name="alertfields"></a><H2>Alert Fields</H2> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shall we link these via id instead of separate anchors?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sory, I'm not sure what you mean..
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Like we've done for scan rules in the help/site:
<H2 id="alertfields">Alert Fields</H2>
addOns/help/src/main/javahelp/contents/start/features/alerts.html
Outdated
Show resolved
Hide resolved
addOns/help/src/main/javahelp/contents/start/features/alerts.html
Outdated
Show resolved
Hide resolved
addOns/help/src/main/javahelp/contents/start/features/alerts.html
Outdated
Show resolved
Hide resolved
Signed-off-by: Simon Bennetts <psiinon@gmail.com>
b1f5f34 to
06331ba
Compare
|
Thank you! |

No description provided.