minor: merge train#214
Merged
wkaisertexas merged 26 commits intomainfrom Aug 16, 2025
Merged
Conversation
- Add visibility parameter to upload functions with options: everyone, friends, only_you - Implement visibility dropdown selector in upload process - Add --visibility CLI flag for command-line uploads - Update VideoDict type to include visibility field - Add example scripts demonstrating private upload usage - Add CLAUDE.md for repository documentation The feature allows users to set video privacy when uploading: - "everyone": Public video (default) - "friends": Visible to friends only (followers you follow back) - "only_you": Private video visible only to the uploader 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Add tests for video dict conversion with visibility parameters - Add validation tests for all visibility options (everyone, friends, only_you) - Add unit tests for upload_video function with visibility parameter passing - Add mocked tests for dropdown interaction in _set_visibility function - Add type validation tests for VideoDict with visibility field Tests ensure proper handling of private video upload functionality across all layers of the application. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Replace bare with specific exception types to follow linting best practices and improve error handling specificity. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Apply consistent code formatting to all modified files: - examples/private_upload.py - src/tiktok_uploader/upload.py - tests/test_upload.py 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Fix mypy type errors in tests and examples by properly typing variables - Remove inappropriate use of type: ignore comments - Update CLAUDE.md with complete CI workflow matching GitHub Actions - Add proper type annotations for VideoDict usage - Ensure all CI checks pass: linting, formatting, type checking, and tests This commit ensures the codebase passes the complete CI pipeline that matches the GitHub Actions workflow. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Through testing, identified that only one selector actually works: //div[@ROLE='option' and contains(., 'Only you')] Removed 40+ lines of complex fallback logic and multiple selector attempts. The function now uses only the verified working selector, making it faster, cleaner, and more maintainable. - Removed unused selectors that never found elements - Removed unnecessary fallback logic - Kept same behavior and interface - All tests still pass - Verified working through actual TikTok uploads 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
feat: add private video upload capability
Add: Ability to upload custom cover
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
multiple merges so the original authors did not have to make edits