-
Notifications
You must be signed in to change notification settings - Fork 742
Update Antivirus Warning documentation on performance impact #9818
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 Antivirus Warning documentation on performance impact #9818
Conversation
Added documentation for a new feature that alerts users about performance impacts caused by antivirus scans. Previously, warnings were limited to Microsoft Defender, now, the IDE also notifies users about other antivirus solutions that may affect performance.
Added documentation for a new feature that alerts users about performance impacts caused by antivirus scans. Previously, warnings were limited to Microsoft Defender; now, the IDE also notifies users about other antivirus solutions that may affect performance.
Hi docs, I believe this documentation should not be on content/en/docs/refguide/modeling/app-performance/antivirus-exclusion.md since this is not related to app performance, but is a general advice |
@rodrigocmoreira |
Hi Mark, I think the documents should not be merged, today this document is already referenced by studio pro, so the user can click in a link that will bring him to this documentation page, another thing we need to make sure that this move would not affect older versions that already point to this link. |
I can make a redirect so that the links continue to work. Would that be OK for you? They can only point to the whole doc and not a particular anchor. The other option is to move this document to be under the performance tips but keep the URL. Any preference? |
Hi @MarkvanMents , if it's possible to set up a redirect in the documentation, that would be great. Even merging could work, as long as the redirect guides the user to the correct section. |
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.
@rodrigocmoreira I've moved the documentation to a more appropriate place and updated the text appropriately. Can you check it looks OK, then I can merge it.
Thanks for the PR.
No negative response from @rodrigocmoreira - merging. |
No description provided.