Skip to content

Conversation

@Tim020
Copy link
Contributor

@Tim020 Tim020 commented Jan 6, 2026

No description provided.

Tim020 and others added 5 commits January 6, 2026 01:20
* Add jump to cue button in cue editor

* Add comprehensive unit tests for CueSearchController API

Implements 20 tests covering validation, exact matching (with case-insensitivity), fuzzy matching algorithm, cue type filtering, and edge cases for the new /api/v1/show/cues/search endpoint.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Sonnet 4.5 <noreply@anthropic.com>
…body

Updated DELETE endpoints to follow HTTP best practices by using query parameters
instead of request bodies. This fixes compatibility issues with HTTP clients and
proxies that may strip bodies from DELETE requests.

Backend changes:
- Updated 8 DELETE methods across 6 controllers to use self.get_argument()
- Added explicit int() conversion with try/except for proper error handling
- Returns 400 error for invalid (non-numeric) IDs

Frontend changes:
- Updated 9 DELETE API calls to use URLSearchParams
- Follows existing pattern from DELETE_SCRIPT_REVISION

All 200 tests passing.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Sonnet 4.5 <noreply@anthropic.com>
…(UTC) (#813)

This commit addresses issue #811 by replacing all instances of the deprecated
datetime.utcnow() method with datetime.now(UTC) to use timezone-aware datetime
objects as recommended by Python 3.11+.

Changes:
- Updated 6 occurrences across 4 files
- Added UTC import from datetime module
- All tests pass (219/219)

Files modified:
- server/controllers/api/show/session/sessions.py
- server/controllers/api/show/shows.py
- server/controllers/api/show/script/script.py
- server/controllers/api/show/script/revisions.py

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Sonnet 4.5 <noreply@anthropic.com>
The stage_direction boolean field was removed in commit 08b6424 and
replaced with the line_type enum. Updated 3 production files and 1 test
file that still referenced the old attribute.

Changed all instances of `if line.stage_direction:` to
`if line.line_type != ScriptLineType.DIALOGUE:` to properly filter
out non-dialogue lines (stage directions, cue lines, and spacing).

Files modified:
- server/controllers/api/show/characters.py (line 201)
- server/controllers/api/show/cast.py (line 189)
- server/utils/show/mic_assignment.py (line 127)
- server/test/utils/show/test_mic_assignment.py (multiple mocks)

Fixes #812

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Sonnet 4.5 <noreply@anthropic.com>
* Add session tag model, schema, migration and CRUD endpoints
* Add front end configuration of session tags
@Tim020 Tim020 added the release Pull requests for creating a new release label Jan 6, 2026
@github-actions github-actions bot added client Pull requests changing front end code server Pull requests changing back end code xlarge-diff labels Jan 6, 2026
@github-actions
Copy link

github-actions bot commented Jan 6, 2026

Python Test Results

  1 files    1 suites   19s ⏱️
253 tests 253 ✅ 0 💤 0 ❌
258 runs  258 ✅ 0 💤 0 ❌

Results for commit 845fcb1.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Jan 6, 2026

Client Test Results

34 tests   34 ✅  0s ⏱️
 1 suites   0 💤
 1 files     0 ❌

Results for commit 845fcb1.

♻️ This comment has been updated with latest results.

@Tim020 Tim020 marked this pull request as ready for review January 6, 2026 23:08
@Tim020 Tim020 enabled auto-merge January 6, 2026 23:08
@Tim020 Tim020 merged commit cb235b1 into main Jan 6, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

client Pull requests changing front end code release Pull requests for creating a new release server Pull requests changing back end code xlarge-diff

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants