Add a list of Windows apps with known compatibility issues #8
Conversation
ameshkov
left a comment
ameshkov
left a comment
There was a problem hiding this comment.
I don't like this format, check how it's done in Android. We may need structured data in the future.
You should also add the description to README
| { | ||
| "executable_name": "Discord.exe", | ||
| "private_issue_id": "AG-43457", | ||
| "public_issue_url": "https://github.com/AdguardTeam/AdguardForWindows/issues/5524" |
There was a problem hiding this comment.
Может ещё краткое описание проблемы ещё указывать? В слаке вроде видел предложение, чтобы в интерфейсе можно было выводить.
There was a problem hiding this comment.
тут несколько проблем
- переводы
- полное обоснование почти нельзя никогда будет дать, но будет ссылка на эту репу, где будет уже ссылка на задачу
There was a problem hiding this comment.
Может типовые статусы последствий тогда? Например, "роутинг мимо Адгарда" (полное отсутствие фильтрации, но будет ли фильтровать DNS - не знаю), "отсутствие фильтрации HTTPS" (возможна фильтрация DNS и применение правил для блокировки доменов) + влияние на блокировку рекламы, если она есть в проге.
Обоснования тоже можно парой-тройкой универсальных фразам сделать. Обычно же поломка проги либо из-за ссл пиннинга или роутинга через блокер (на дроиде такое бывает).
Если сделать универсальными, то можно будет айдишниками строк указывать в жсоне.
И в интерфейсе в можно будет увидеть краткие подробности. А если юзер захочет подробности увидеть - то пойдёт на гитхаб.
There was a problem hiding this comment.
Типовые последствия следуют из того в каком оно файле
| @@ -0,0 +1 @@ | |||
| // List of browsers supported by AdGuard No newline at end of file | |||
There was a problem hiding this comment.
Shouldn't it be a JSON file?
As far as I understand a single browser may have several executables
| { | ||
| "executable_name": "Discord.exe", | ||
| "private_issue_id": "AG-43457", | ||
| "public_issue_url": "https://github.com/AdguardTeam/AdguardForWindows/issues/5524" |
There was a problem hiding this comment.
Типовые последствия следуют из того в каком оно файле
…t decided on yet.
…ner.exe to routing_exclusions.json
…ting_exclusions.json
| ## What AdGuard applications use these filtering lists? | ||
|
|
||
| Currently, they apply to AdGuard for Android. | ||
| Currently, they apply to AdGuard for Android and AdGuard for Windows. |
There was a problem hiding this comment.
для макоси и клишных блокеров не актуально?
There was a problem hiding this comment.
Пока нет, так как в них еще не реализован System wide filtering
…ing_exclusions.json
…ing_exclusions.json
…ing_exclusions.json
…uardTeam/AdguardForWindows#5721 to routing_exclusions.json
…ing_exclusions.json
…ing_exclusions.json
…ing_exclusions.json
Add a list of Windows apps with known compatibility issues