xiaoxia/xiaoxia-saas
1
0
Fork 0
Code Issues 14 Pull Requests 2 Actions Packages Projects Releases 7 Wiki Activity

fix(clone): 克隆音色「去配音库上传」跳转修复 #1652

Merged
xiaoxia merged 2 commits from fix/clone-modal-upload-redirect into develop 2026-09-03 17:29:53 +08:00
Conversation 6 Commits 2 Files Changed 2
+7 -3
xiaoxia commented 2026-09-03 17:08:31 +08:00
Owner
Copy Link
Copy Source

问题

克隆音色弹窗中点击「去配音库上传」跳转到 /app/voice-materials（多余的旧配音素材页面）。

修复

改为跳转到 /app/voices?tab=material&upload=1，即配音库页面素材Tab，并自动打开上传弹窗。

与 Step5VoiceSelect 的「去配音库上传」行为保持一致。

## 问题 克隆音色弹窗中点击「去配音库上传」跳转到 `/app/voice-materials`（多余的旧配音素材页面）。 ## 修复 改为跳转到 `/app/voices?tab=material&upload=1`，即配音库页面素材Tab，并自动打开上传弹窗。 与 Step5VoiceSelect 的「去配音库上传」行为保持一致。
gitea-actions bot commented 2026-09-03 17:10:18 +08:00
Copy Link
Copy Source

🚀 预览环境已部署

项目 详情
PR号 #1652
预览链接 https://pr-1652.preview.xiaoxiajianji.com
API环境 staging

💡 预览环境使用 staging API 数据，请勿在预览环境中操作重要数据。

🔄 每次提交新代码后预览环境会自动更新。

🗑️ PR 关闭或合并后，预览环境会自动清理。

🚀 **预览环境已部署** | 项目 | 详情 | |------|------| | PR号 | #1652 | | 预览链接 | [https://pr-1652.preview.xiaoxiajianji.com](https://pr-1652.preview.xiaoxiajianji.com) | | API环境 | staging | > 💡 预览环境使用 staging API 数据，请勿在预览环境中操作重要数据。 > > 🔄 每次提交新代码后预览环境会自动更新。 > > 🗑️ PR 关闭或合并后，预览环境会自动清理。
auto-approve-bot approved these changes 2026-09-03 17:10:29 +08:00
auto-approve-bot left a comment
Collaborator
Copy Link
Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。
auto-approve-bot approved these changes 2026-09-03 17:10:29 +08:00
auto-approve-bot left a comment
Collaborator
Copy Link
Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。
auto-approve-bot commented 2026-09-03 17:22:08 +08:00
Collaborator
Copy Link
Copy Source

Auto merge skipped after multiple 405 errors: PR may have conflicts or unresolved checks. Please review manually. This is not a CI failure.

Auto merge skipped after multiple 405 errors: PR may have conflicts or unresolved checks. Please review manually. This is not a CI failure.
xiaoxia added 2 commits 2026-09-03 17:26:20 +08:00
fix(clone): 克隆音色「去配音库上传」改为跳转到配音库素材Tab并打开上传弹窗 ef08ee1a4c
fix(voices): 修复 upload=1 参数未清理导致 Tab 切换被强制回 material 的问题
CI/CD Pipeline / Check push changed paths (pull_request) Has been skipped
Details
CI/CD Pipeline / Check if frontend-only change (pull_request) Successful in 2s
Details
CI/CD Pipeline / Dedup Check - skip PR tests when covered by push pipeline (pull_request) Successful in 3s
Details
CI/CD Pipeline / Integration Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / Unit Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / PR Build Worker Image (pull_request) Has been skipped
Details
CI/CD Pipeline / PR Build API Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Staging API Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Staging Web Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Staging Worker Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Retag skipped Staging API Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Retag skipped Staging Web Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Retag skipped Staging Worker Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Deploy Staging (Watchtower auto-deploy) (pull_request) Has been skipped
Details
CI/CD Pipeline / Staging E2E Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / Staging API Integration Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / ACR Image Cleanup (pull_request) Has been skipped
Details
CI/CD Pipeline / PR Build Web Image (pull_request) Successful in 34s
Details
AI Code Review / AI Code Review (pull_request) Successful in 1m8s
Details
CI/CD Pipeline / Frontend Unit Tests (pull_request) Successful in 1m21s
Details
CI/CD Pipeline / Frontend Lint (pull_request) Successful in 1m34s
Details
Preview Deploy / Deploy Preview Environment (pull_request) Successful in 1m35s
Details
PR Automation / Auto Approve on CI Green (pull_request) Successful in 1m37s
Details
CI/CD Pipeline / Validate - Python (mypy + alembic) (pull_request) Successful in 1m48s
Details
CI/CD Pipeline / Validate - Style (pull_request) Successful in 2m9s
Details
PR Automation / Auto Merge on CI Green + Approved (pull_request) Successful in 1m46s
Details
ACR Cleanup / ACR Image Cleanup (pull_request_target) Successful in 8s
Details
Preview Cleanup / Cleanup Preview Environment (pull_request) Successful in 32s
Details
CI/CD Pipeline / Validate - Security (pull_request) Successful in 6m22s
Details
CI/CD Pipeline / Build Production API Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Production Web Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Production Worker Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Canary Release to Production (pull_request) Has been skipped
Details
CI/CD Pipeline / Deploy Production (pull_request) Has been skipped
Details
CI/CD Pipeline / Production Browser E2E (pull_request) Has been skipped
Details
CI/CD Pipeline / CI Gate (pull_request) Successful in 3s
Details
bba22483c8
xiaoxia force-pushed fix/clone-modal-upload-redirect from 961c067828 to bba22483c8 2026-09-03 17:26:20 +08:00 Compare
auto-approve-bot commented 2026-09-03 17:27:33 +08:00
Collaborator
Copy Link
Copy Source

【阻塞级判定】

  • 是否存在阻塞级问题：否
  • 阻塞级问题数量：0 个

📊 审查概览

  • 整体评价：通过
  • 建议级问题数量：0 个

🔴 阻塞级问题（必须修复）

无

💡 改进建议（不阻塞合并）

无

✅ 良好实践

  1. [VoiceLibrary.tsx] 一次性触发器逻辑
    • 具体内容：在 useEffect 中检测到 upload 参数并执行了相应逻辑后，立即通过 setSearchParams 清除该参数。这种“触发即清理”的模式有效避免了后续组件重渲染或 Tab 切换时重复触发弹窗，是处理 URL 参数触发状态变更的最佳实践。
  2. [VoiceLibrary.tsx] 使用 replace: true
    • 具体内容：在更新 URL 参数时使用了 { replace: true } 选项，这样不会在浏览器历史记录中堆砌无意义的中间状态，用户体验更好。

🤖 由 AI 代码审查机器人自动生成 | 2026-09-03 09:27:32 | 模型:

### 【阻塞级判定】 - 是否存在阻塞级问题：否 - 阻塞级问题数量：0 个 ### 📊 审查概览 - 整体评价：通过 - 建议级问题数量：0 个 ### 🔴 阻塞级问题（必须修复） 无 ### 💡 改进建议（不阻塞合并） 无 ### ✅ 良好实践 1. **[VoiceLibrary.tsx] 一次性触发器逻辑** - 具体内容：在 `useEffect` 中检测到 `upload` 参数并执行了相应逻辑后，立即通过 `setSearchParams` 清除该参数。这种“触发即清理”的模式有效避免了后续组件重渲染或 Tab 切换时重复触发弹窗，是处理 URL 参数触发状态变更的最佳实践。 2. **[VoiceLibrary.tsx] 使用 replace: true** - 具体内容：在更新 URL 参数时使用了 `{ replace: true }` 选项，这样不会在浏览器历史记录中堆砌无意义的中间状态，用户体验更好。 --- <sub>🤖 由 AI 代码审查机器人自动生成 | 2026-09-03 09:27:32 | 模型: </sub> <!-- AI_CODE_REVIEW_AUTO_COMMENT -->
xiaoxia merged commit bb7dc71da3 into develop 2026-09-03 17:29:53 +08:00
xiaoxia deleted branch fix/clone-modal-upload-redirect 2026-09-03 17:29:53 +08:00
gitea-actions bot commented 2026-09-03 17:30:14 +08:00
Copy Link
Copy Source

🗑️ 预览环境已清理

PR #1652 已关闭或合并，对应的预览环境已被清理。

如有需要，可以重新打开 PR 来重新生成预览环境。

🗑️ **预览环境已清理** PR #1652 已关闭或合并，对应的预览环境已被清理。 > 如有需要，可以重新打开 PR 来重新生成预览环境。
Sign in to join this conversation.
Reviewers
No Reviewers
auto-approve-bot
Labels
Clear labels
P0
阻塞性，必须立即处理
 P1
高优先级
 P2
中优先级
 P3
低优先级/后考虑
 architecture
架构设计
 backend
后端相关
 blocked
阻塞中
 bug
Bug修复
 ci-bug
CI本身的bug
 ci-optimization
CI优化类任务
 ci-pipeline
CI流水线/workflow相关
 cleanup
代码清理
 devops
运维/部署/CI
 documentation
文档相关
 feature
新功能
 frontend
前端相关
 question
需要讨论确认
 ready
PR CI通过，可合并
 refactor
代码重构
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
auto-approve-bot xiaoxia
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#1652
Delete Branch "fix/clone-modal-upload-redirect"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?