Skip to content

add-info-about-filtering-in-AGW#893

Open
el-termikael wants to merge 5 commits into
masterfrom
AG-56324
Open

add-info-about-filtering-in-AGW#893
el-termikael wants to merge 5 commits into
masterfrom
AG-56324

Conversation

@el-termikael

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions

Copy link
Copy Markdown

Preview was deployed to: https://pull-request-893.kb-adg.pages.dev/

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR expands the AdGuard for Windows v8.0 App management documentation to explain the app routing/filtering model (compatibility list, exclusion lists, and handling of unknown apps), and adds a dedicated section describing the Filter unknown apps setting.

Changes:

  • Added a new section describing how routing, filtering, and HTTPS filtering decisions are made (compatibility list vs. exclusion lists vs. unknown apps).
  • Added a new Filter unknown apps section with UI screenshot and behavior breakdown.
  • Removed an older app-management screenshot near the top of the page.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +64 to +66
import Tabs from '@theme/Tabs';

To ensure the proper operation of all apps installed on your device, we recommend routing only problem-free apps through AdGuard.
import TabItem from '@theme/TabItem';
To ensure the proper operation of all apps installed on your device, we recommend routing only problem-free apps through AdGuard.
import TabItem from '@theme/TabItem';

:::tip[There are three categories of apps:]
Comment thread docs/adguard-for-windows-8/app-management/app-management.md Outdated
Comment thread docs/adguard-for-windows-8/app-management/app-management.md Outdated
@github-actions

Copy link
Copy Markdown

Preview was deployed to: https://pull-request-893.kb-adg.pages.dev/

el-termikael and others added 2 commits July 13, 2026 17:01
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@github-actions

Copy link
Copy Markdown

Preview was deployed to: https://pull-request-893.kb-adg.pages.dev/

1 similar comment
@github-actions

Copy link
Copy Markdown

Preview was deployed to: https://pull-request-893.kb-adg.pages.dev/

@github-actions

Copy link
Copy Markdown

Preview was deployed to: https://pull-request-893.kb-adg.pages.dev/

Comment thread docs/adguard-for-windows-8/app-management/app-management.md
Comment thread docs/adguard-for-windows-8/app-management/app-management.md Outdated

:::info

The compatibility lists are continuously updated — and you can help improve them. If you notice that an app does not work correctly with filtering enabled, report it [on GitHub](https://github.com/AdguardTeam/CompatibilityIssues) or contact support via the app. Your feedback helps improve AdGuard for everyone.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe make it bold - an app does not work correctly with filtering enabled

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd rather make CTA ("report it on GitHub") bold, but admonitions are already highlighting the text, aren't they?

@el-termikael el-termikael requested a review from adbuker July 13, 2026 14:25

The *Filter unknown apps* setting allows you to choose between two modes — a safe default one where only trusted apps are filtered and a manual one where you decide which apps to filter. It affects only new and unknown applications.

There are three categories of apps:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

there was a nice table in the task, maybe we can use a table here rather than a list?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

also, from the subheading of the list, it’s not entirely obvious what these categories are for. It might be worth explaining this more clearly:
Here is a quick overview of how different app categories behave depending on this setting:


![Filter unknown apps *mobile](https://cdn.adtidy.org/content/kb/ad_blocker/windows/version_8/app_management/app-management.png)

The *Filter unknown apps* setting allows you to choose between two modes — a safe default one where only trusted apps are filtered and a manual one where you decide which apps to filter. It affects only new and unknown applications.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is there a way to convey it more clearly? my understanding was that the Filter unknown apps option is enabled by default and determines how AdGuard handles new apps that are not yet present in the compatibility or exclusion lists. and then we can describe each scenario:

  • when enabled: AdGuard automatically routes traffic...
  • when disabled: You will need to manually configure routing for new apps...


:::info

The compatibility lists are continuously updated — and you can help improve them. If you notice that an app does not work correctly with filtering enabled, report it [on GitHub](https://github.com/AdguardTeam/CompatibilityIssues) or contact support via the app. Your feedback helps improve AdGuard for everyone.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we usually exclude prepositions from links

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.

4 participants