Release 0.183.0#2982
Conversation
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
* [pre-commit.ci] pre-commit autoupdate updates: - [github.com/scop/pre-commit-shfmt: v3.12.0-2 → v3.13.0-1](scop/pre-commit-shfmt@v3.12.0-2...v3.13.0-1) - [github.com/astral-sh/ruff-pre-commit: v0.14.14 → v0.15.9](astral-sh/ruff-pre-commit@v0.14.14...v0.15.9) * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
* feat: added celery beat for large number of videos in youtube * feat: added weekend scheduling and documented command usage * chore: removed manual batching params and updated docs * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * refactor: updated update_youtube_tags_batch to reduce complexity * chore: addressed feedback * chore: spanned retries across the day if quota error * refactor: added constants, deduplicated methods and added utils --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
* feat: handled refrencing_content for captions/transcripts and added tests for navmenu objects * chore: adde null handling for navmenu * chore: handled existing caption/transcript files * chore: addressed feedback from copilot and seer * chore: replaced hardcoded values with settings
There was a problem hiding this comment.
Code Review
This pull request introduces a new management command and Celery task for asynchronously updating YouTube video tags, with features like API quota management, scheduled processing, and batch fetching of video snippets. It also significantly refactors the handling of WebsiteContent references, particularly for video captions/transcripts and navigation menus, ensuring these relationships are correctly tracked. Additionally, several Python and JavaScript dependencies have been updated. A review comment suggests improving the get_dict_field utility function to handle None values for the initial object to prevent potential AttributeError.
| return current_obj.get(fields[-1]) | ||
|
|
There was a problem hiding this comment.
This is a good improvement to make the function more robust against non-dict intermediate values. However, the function can still raise an AttributeError if the initial obj is None, which is a possible value for WebsiteContent.metadata. A more robust approach would be to check for a falsy obj at the beginning of the function.
Umar Hassan
renovate[bot]
pre-commit-ci[bot]