Skip to content

Some FE improvements - #224

Merged
itdoginfo merged 9 commits into
mainfrom
feat/fe-improvements
Oct 23, 2025
Merged

itdoginfo merged 9 commits into
mainfrom
feat/fe-improvements

Conversation

@divocat

@divocat divocat commented Oct 23, 2025

Copy link
Copy Markdown
Collaborator
  1. Добавлен wiki блок в диагностику
  2. Остатки по диагностике
  3. Повышен shell timeout до 15с для слабых устройств
  4. Кастом порт и путь (для айпи) для dns
  5. Переводы для проверок диагностики и системных виджетов
  6. Поправлена proxy link валидация убираем лишние пробелы в конце и разрешены слеши после порта
  7. На диагностике если не удалось получить глобал чек или конфиг или логи - будет уведомление об ошибке
  8. Поправлены стили для уведомления о некорректно выбранных списках

Copilot AI review requested due to automatic review settings October 23, 2025 17:39
@divocat
divocat requested a review from itdoginfo as a code owner October 23, 2025 17:39

Copilot AI left a comment

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.

Pull Request Overview

This PR enhances the frontend diagnostics interface and improves various validation functions. The main purpose is to make diagnostic checks more user-friendly by consolidating repetitive messages, adding visual guidance through a wiki troubleshooting widget, and supporting DNS configurations with custom ports and paths.

Key Changes:

  • Consolidated diagnostic check messages to reduce translation overhead and improve consistency
  • Added a wiki troubleshooting widget that displays based on diagnostic check results
  • Enhanced DNS validation to support custom ports and paths (e.g., 1.1.1.1:853/dns-query)
  • Improved proxy URL validation to handle trailing whitespace and slashes after ports
  • Increased shell command timeout from 10s to 15s for slower devices

Reviewed Changes

Copilot reviewed 28 out of 28 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
luci-app-podkop/po/templates/podkop.pot Updated translation template with consolidated messages and new troubleshooting strings
luci-app-podkop/po/ru/podkop.po Updated Russian translations matching the template changes
luci-app-podkop/htdocs/luci-static/resources/view/podkop/section.js Removed CSS class from warning notification element
luci-app-podkop/htdocs/luci-static/resources/view/podkop/main.js Bundled JS with all frontend changes including validation improvements
fe-app-podkop/src/validators/validateVlessUrl.ts Added port cleaning to handle trailing slashes in VLESS URLs
fe-app-podkop/src/validators/validateProxyUrl.ts Added URL trimming to handle leading/trailing whitespace
fe-app-podkop/src/validators/validateDns.ts Enhanced to support port and path specifications in DNS addresses
fe-app-podkop/src/validators/tests/validateDns.test.js Added test cases for DNS validation with ports and paths
fe-app-podkop/src/podkop/tabs/diagnostic/* Multiple diagnostic improvements including wiki widget and consolidated check messages
fe-app-podkop/src/icons/renderBookOpenTextIcon24.ts New icon component for the wiki troubleshooting widget
fe-app-podkop/locales/* Updated locale files with new and consolidated translation strings

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread fe-app-podkop/src/podkop/tabs/diagnostic/styles.ts
Comment thread fe-app-podkop/src/podkop/tabs/diagnostic/checks/runDnsCheck.ts
Comment thread luci-app-podkop/po/ru/podkop.po Outdated

@itdoginfo itdoginfo left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

LGTM

@itdoginfo
itdoginfo merged commit c3fac99 into main Oct 23, 2025
1 check passed
@itdoginfo
itdoginfo deleted the feat/fe-improvements branch October 23, 2025 18:14
Dushnilin referenced this pull request in Dushnilin/tachyon Jul 17, 2026
Screamshow pushed a commit to Screamshow/forkop-x that referenced this pull request Sep 3, 2026
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.

3 participants