Skip to content

fix(tasks): correct task path links - #743

Open
Sianao wants to merge 2 commits into
OpenListTeam:mainfrom
Sianao:main
Open

Sianao wants to merge 2 commits into
OpenListTeam:mainfrom
Sianao:main

Conversation

@Sianao

@Sianao Sianao commented Nov 14, 2026

Copy link
Copy Markdown

Summary / 摘要

  • Add the configured site base path to links in task details, so paths navigate correctly when OpenList is served under a subpath.

  • Check user base paths at directory boundaries, so /team2/file is not treated as a path under /team.

  • The shared path helper is used by copy, move, upload, offline transfer, and decompression tasks.

  • This PR has breaking changes.

  • This PR changes public API, config, storage format, or migration behavior.

  • This PR requires corresponding changes in related repositories.

Testing / 测试

  • Checked the patch with git diff --check.
  • Ran ad hoc Node checks for site base paths and user path boundaries.
  • Did not run a full frontend build or browser test; dependencies were not installed locally.
  • go test ./... is not applicable to this frontend change.
  • go test ./...
  • Manual test / 手动测试:

Checklist / 检查清单

  • I have read CONTRIBUTING.
  • I confirm this contribution follows the repository license, contribution policy, and code of conduct.
  • I have formatted the changed code with prettier where applicable.
  • I have requested review from relevant maintainers or code owners where applicable.

AI Disclosure / AI 使用声明

  • This PR includes AI-assisted content.

Tools used / 使用工具:

  • Codex

Usage scope / 使用范围:

  • Code generation / 代码生成

  • Review assistance / 审查辅助

  • I have reviewed and validated all AI-assisted content included in this PR.

  • I have ensured that all AI-assisted commits include Co-Authored-By attribution.

  • I can reproduce all AI-assisted content included in this PR without any AI tools.

- Prefix task links with the configured site base path.
- Match the user base path as a complete directory prefix.
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.

1 participant