Skip to content

Feature/batch disk label#21

Merged
Betanu701 merged 10 commits intofeature-utilize-dvd-label-batch-processingfrom
feature/Batch-Disk-Label
Oct 19, 2025
Merged

Feature/batch disk label#21
Betanu701 merged 10 commits intofeature-utilize-dvd-label-batch-processingfrom
feature/Batch-Disk-Label

Conversation

@Betanu701
Copy link
Copy Markdown
Owner

This pull request introduces a new batch rename feature for TV series disc folders, including both backend API endpoints and frontend UI integration. It adds a dedicated migration for tracking batch rename history, new configuration options, and updates to the navigation and settings. The changes are grouped below by theme.

Batch Rename Feature Implementation

  • Adds a new API endpoint /batch_rename to support previewing, executing, and rolling back batch rename operations for TV series disc folders, as well as retrieving recent batch operations. (arm/ui/jobs/jobs.py)
  • Introduces a new UI blueprint and route (/batch_rename_view) for the batch rename feature, displaying completed jobs and allowing users to initiate batch renames. (arm/ui/batch_rename_ui/batch_rename_ui.py, arm/ui/batch_rename_ui/__init__.py, arm/ui/__init__.py, arm/ui/templates/nav.html) [1] [2] [3] [4] [5]
  • Adds a new Alembic migration to create the batch_rename_history table, which tracks all batch rename operations and their outcomes. (arm/migrations/versions/c3d4e5f6g7h8_add_batch_rename_history.py)

Custom Metadata Lookup for Batch Operations

  • Adds /batch_custom_lookup API endpoint to allow searching for titles in TMDB/OMDb and applying custom identification metadata to multiple jobs at once. (arm/ui/jobs/jobs.py)

Configuration and UI Enhancements

  • Introduces new configuration options for batch rename naming style, zero-padding, and consolidation defaults, with documentation for each. (arm/ui/comments.json)
  • Updates the navigation bar to include a link to the new Batch Rename UI. (arm/ui/templates/nav.html)
  • Minor UI tweaks for alignment and spacing in the database view. (arm/ui/database/templates/databaseview.html) [1] [2]

@Betanu701 Betanu701 requested a review from Copilot October 19, 2025 21:30
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request introduces a comprehensive batch rename feature for TV series disc folders, enabling users to rename multiple completed rips at once with consistent disc-label-based naming. The feature includes preview capabilities, series detection, conflict resolution, rollback functionality, and a complete audit trail.

Key Changes:

  • Backend API endpoints for batch rename operations (preview, execute, rollback) and custom metadata lookup
  • New database migration for tracking batch rename history with rollback support
  • Dedicated UI pages with multi-step modals for batch operations
  • Configuration defaults for naming style, zero-padding, and consolidation preferences

Reviewed Changes

Copilot reviewed 16 out of 16 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
setup/arm.yaml Adds three new configuration options for batch rename defaults
arm/ui/comments.json Adds documentation for the new batch rename configuration options
arm/ui/templates/nav.html Adds navigation link to new Batch Rename UI
arm/ui/__init__.py Registers new batch_rename_ui blueprint
arm/ui/jobs/jobs.py Implements /batch_rename and /batch_custom_lookup API endpoints
arm/ui/batch_rename.py Core logic for preview, execute, and rollback operations
arm/ui/batch_rename_ui/batch_rename_ui.py Flask route for batch rename view page
arm/ui/batch_rename_ui/__init__.py Blueprint initialization
arm/ui/batch_rename_ui/templates/batch_rename_view.html Full UI template with modals for batch operations
arm/ui/static/js/database_batch_rename.js JavaScript for batch rename in database view
arm/ui/static/js/batch_rename_page.js JavaScript for dedicated batch rename page
arm/migrations/versions/c3d4e5f6g7h8_add_batch_rename_history.py Database migration for batch_rename_history table
arm/ui/database/templates/databaseview.html Minor CSS fix for card header alignment
docs/DEMO-Batch-Rename.md Step-by-step demonstration guide
arm_wiki/Batch-Rename-TV-Series.md Comprehensive feature documentation
arm_wiki/_Sidebar.md Adds links to new feature documentation
Comments suppressed due to low confidence (1)

arm/ui/batch_rename_ui/templates/batch_rename_view.html:1

  • The poster URL from API response is directly used as image src without validation. This could lead to loading malicious resources. Validate the URL is from an expected domain or use a proxy to serve external images.
{% extends "base.html" %}

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

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@Betanu701 Betanu701 merged commit c397ce7 into feature-utilize-dvd-label-batch-processing Oct 19, 2025
3 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.

2 participants