Skip to content

fix(navigation): 修正前台反馈、登录回跳与来源链接 - #224

Merged
fxbin merged 3 commits into
mainfrom
issue-223-user-navigation
Nov 3, 2026
Merged

fxbin merged 3 commits into
mainfrom
issue-223-user-navigation

Conversation

@fxbin

@fxbin fxbin commented Nov 3, 2026

Copy link
Copy Markdown
Owner

用户点击“反馈建议”或个人反馈历史会进入后台工作台,公共来源标注与来源收藏也存在跳向后台或无关私有信源列表的问题。新增前台 /feedback 提交/我的历史/处理回复页面,取消旧反馈重定向及重复弹窗;来源仅在有明确原文时提供链接,模块关闭时不再展示不可达榜单入口。

登录守卫及需要登录的操作保留原路径和查询;已登录但权限/功能不可用时返回前台。OAuth失败重试保留原目标,回跳校验拒绝控制字符等可被浏览器归一化为站外的输入。趋势与小说榜单8处缺少原文URL的伪链接改为文字。

Closes #223

分支 issue-223-user-navigation;反馈/登录、来源链接及长期合同分为三个提交。长期维护入口为 .agents/notes/implemented/feature/2026-11-03-user-navigation-contract.md,截图、fixture与运行证据仅留本地 .vidt/;AGENTS稳定约定未变。

Verification

  • cd frontend && npm run test:coverage:59文件441项通过;包含个人历史仅用mine接口、角色/模块入口、回跳安全与重试、原文URL缺失回归。修复前已复现2项失败。
  • npx tsc --noEmit、npm run build、npm run test:navigation、notes校验和 git diff --check 通过。
  • 改动文件ESLint的10条error均为AuthProvider预存react-hooks/refs,与基线逐条规则/级别/消息一致;新逻辑无新增error,未降低门禁。既有unused/hook warnings未混入修复。
  • 盘点214个非测试TS/TSX文件中的110个链接/显式导航位置,并核对导航配置和重定向;管理后台及管理更新仍限定管理员,历史管理路由保留。
  • 真实浏览器使用隔离普通用户fixture:游客登录回反馈页、提交失败保留草稿/重试成功、个人历史翻页/回复/失败重试、更新页入口、后台拒绝与模块关闭去向、收藏来源外链、OAuth失败重试及390px无横向溢出均通过。

边界

本轮只改前端及长期合同,未改后端权限/schema,未运行backend pytest。浏览器提交只写隔离内存服务,不写业务库;不代表真实账号隔离、生产反馈交付或真实OAuth提供方验收。未逐一验证第三方外链和真实设备。此次合并不含额外生产操作。

独立最终复核:/root/navigation_final_verify 对 f51fc01..9981c09 的28文件范围通过,无阻塞问题,原审计意见已关闭;长期owning note职责与持续维护价值已核对。未重复全量跑测,测试/构建结论核阅本地日志。

fxbin added 3 commits November 3, 2026 20:14
前台反馈统一使用提交与我的历史页面;登录守卫、需要登录的动作及OAuth失败重试保留原目标,拒绝可归一化为站外地址的回跳输入。
公共来源与私有信源区分;来源收藏保留原文,模块关闭时隐藏榜单入口,缺少有效原文URL时显示文字。
@fxbin
fxbin merged commit c1436e7 into main Nov 3, 2026
8 checks passed
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