-
Notifications
You must be signed in to change notification settings - Fork 316
add adguard for windows v8 section #785
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: master
Are you sure you want to change the base?
Conversation
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.
Other comments (15)
- docs/adguard-for-windows-8/protection/protection.md (16-16) The text mentions the 'AdGuard Assistant browser extension' but doesn't provide a link to more information about it, unlike the other features that have dedicated article links.
- docs/adguard-for-windows-8/protection/protection.md (18-28) The links to feature articles (lines 18-28) appear to be using a relative path format with `kb/` prefix. Depending on your documentation system, these might need to be adjusted to ensure they work correctly. Consider using full paths or the appropriate link format for your documentation system.
- docs/adguard-for-windows-8/settings/settings.md (14-14) The introductory text "There are other useful AdGuard options that shouldn't go unnoticed in this article, since they add much to user experience" is vague. Consider providing a clearer overview of what the Settings section contains, such as "The Settings section provides access to customize AdGuard's appearance, update behavior, license management, and more."
-
docs/adguard-for-windows-8/protection/ad-blocking.md (23-23)
There's an inconsistency in the bullet point formatting. The fourth item is missing a comma at the end, unlike the other items in the list.
- enable automatic activation of language-specific filters, -
docs/adguard-for-windows-8/settings/app-settings/_category_.json (1-6)
This file is missing a newline at the end. It's a good practice to end all text files with a newline character.
{ "position": 1, "label": "App settings", "collapsible": true, "collapsed": true } -
docs/adguard-for-windows-8/settings/app-settings/advanced-settings.md (14-14)
There's a typo in this sentence - 'AdGuard for Windows AdGuard' should probably be just 'AdGuard for Windows'.
The *Advanced settings* section contains options that go beyond what most users usually need. AdGuard for Windows works perfectly out of the box, but these settings can be useful in special cases or for troubleshooting. -
docs/adguard-for-windows-8/protection/browsing-security.md (28-28)
There's a duplicate word 'security' in this bullet point.
- allow anonymous security-related data to be sent to AdGuard, which helps us keep our database of harmful websites up to date. -
docs/adguard-for-windows-8/settings/_category_.json (4-6)
This file is missing a newline at the end. It's a good practice to ensure all text files end with a newline character.
"collapsible": true, "collapsed": true } -
docs/adguard-for-windows-8/settings/license.md (21-21)
Missing punctuation at the end of this bullet point for consistency with the other list items.
- refresh the license status; -
docs/adguard-for-windows-8/protection/_category_.json (1-6)
This file is missing a newline at the end. It's a good practice to end all text files with a newline character.
{ "position": 2, "label": "Protection", "collapsible": true, "collapsed": true } -
docs/adguard-for-windows-8/protection/parental-control.md (16-16)
There's a typo in the image markdown syntax. The asterisk should be inside the alt text parentheses.
 -
docs/adguard-for-windows-8/protection/parental-control.md (22-22)
There's a typo in the text - 'exe.files' should likely be 'exe files' without the period.
To prevent children from downloading unwanted programs, enable the *Block downloading exe files* option. This option blocks them from downloading and installing software. -
docs/adguard-for-windows-8/statistics/statistics.md (36-36)
This sentence is missing a period at the end, which is inconsistent with the rest of the document.
This section shows data about the requests handled by DNS protection. You can see the total number of requests sent and how many were blocked by AdGuard. -
docs/adguard-for-windows-8/settings/app-settings/advanced-settings.md (46-46)
The option name referenced here doesn't match the actual option name used earlier in the document (line 40). For consistency, they should match.
Since version 7.12, AdGuard doesn't start filtering traffic automatically after you boot your PC — unless the *Open AdGuard at system start-up* option is enabled. Instead, it runs in a passive "idle" mode. -
docs/adguard-for-windows-8/statistics/statistics.md (24-24)
The UI element reference should be consistently formatted (either in italics or with proper formatting) and the sentence should end with a period.
**Open Filtering log** shows the last 10,000 requests processed by AdGuard.
💡 To request another review, post a new comment with "/windsurf-review".
Co-authored-by: windsurf-bot[bot] <189301087+windsurf-bot[bot]@users.noreply.github.com>
…om/AdguardTeam/KnowledgeBase into add-adguard-for-windows-v8-section
|
Preview was deployed to: https://pull-request-785.kb-adg.pages.dev/ |
Co-authored-by: Helen <58733007+el-termikael@users.noreply.github.com>
Co-authored-by: Helen <58733007+el-termikael@users.noreply.github.com>
Co-authored-by: Helen <58733007+el-termikael@users.noreply.github.com>
Co-authored-by: Helen <58733007+el-termikael@users.noreply.github.com>
|
Preview was deployed to: https://pull-request-785.kb-adg.pages.dev/ |
2 similar comments
|
Preview was deployed to: https://pull-request-785.kb-adg.pages.dev/ |
|
Preview was deployed to: https://pull-request-785.kb-adg.pages.dev/ |
Co-authored-by: Helen <58733007+el-termikael@users.noreply.github.com>
Co-authored-by: Helen <58733007+el-termikael@users.noreply.github.com>
|
Preview was deployed to: https://pull-request-785.kb-adg.pages.dev/ |
1 similar comment
|
Preview was deployed to: https://pull-request-785.kb-adg.pages.dev/ |
Co-authored-by: Helen <58733007+el-termikael@users.noreply.github.com>
Co-authored-by: Helen <58733007+el-termikael@users.noreply.github.com>
Co-authored-by: Helen <58733007+el-termikael@users.noreply.github.com>
Co-authored-by: Helen <58733007+el-termikael@users.noreply.github.com>
Co-authored-by: Helen <58733007+el-termikael@users.noreply.github.com>
Co-authored-by: Helen <58733007+el-termikael@users.noreply.github.com>
|
Preview was deployed to: https://pull-request-785.kb-adg.pages.dev/ |
1 similar comment
|
Preview was deployed to: https://pull-request-785.kb-adg.pages.dev/ |
Co-authored-by: Helen <58733007+el-termikael@users.noreply.github.com>
|
Preview was deployed to: https://pull-request-785.kb-adg.pages.dev/ |
6 similar comments
|
Preview was deployed to: https://pull-request-785.kb-adg.pages.dev/ |
|
Preview was deployed to: https://pull-request-785.kb-adg.pages.dev/ |
|
Preview was deployed to: https://pull-request-785.kb-adg.pages.dev/ |
|
Preview was deployed to: https://pull-request-785.kb-adg.pages.dev/ |
|
Preview was deployed to: https://pull-request-785.kb-adg.pages.dev/ |
|
Preview was deployed to: https://pull-request-785.kb-adg.pages.dev/ |
Co-authored-by: Helen <58733007+el-termikael@users.noreply.github.com>
Co-authored-by: Helen <58733007+el-termikael@users.noreply.github.com>
…ings.md Co-authored-by: Helen <58733007+el-termikael@users.noreply.github.com>
|
Preview was deployed to: https://pull-request-785.kb-adg.pages.dev/ |
4 similar comments
|
Preview was deployed to: https://pull-request-785.kb-adg.pages.dev/ |
|
Preview was deployed to: https://pull-request-785.kb-adg.pages.dev/ |
|
Preview was deployed to: https://pull-request-785.kb-adg.pages.dev/ |
|
Preview was deployed to: https://pull-request-785.kb-adg.pages.dev/ |
…om/AdguardTeam/KnowledgeBase into add-adguard-for-windows-v8-section
|
Preview was deployed to: https://pull-request-785.kb-adg.pages.dev/ |
|
Preview was deployed to: https://pull-request-785.kb-adg.pages.dev/ |
|
Preview was deployed to: https://pull-request-785.kb-adg.pages.dev/ |
|
Preview was deployed to: https://pull-request-785.kb-adg.pages.dev/ |
No description provided.