mirror of
https://github.com/sigma67/ytmusicapi.git
synced 2026-04-24 23:06:13 +03:00
[PR #662] [CLOSED] Added Lyrics w. Timestamps #695
Labels
No labels
a/b
bug
documentation
enhancement
good first issue
help wanted
invalid
pull-request
question
wontfix
yt-error
yt-update
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/ytmusicapi#695
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
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?
📋 Pull Request Information
Original PR: https://github.com/sigma67/ytmusicapi/pull/662
Author: @heinrich26
Created: 10/10/2024
Status: ❌ Closed
Base:
main← Head:patch-1📝 Commits (10+)
4d4a76fAdded Lyrics w. Timestamps14313beUpdate browsing.py0885616Combined both get_lyrics methods into one and fixed some typechecking errors28a8828Fixed a missing hasTimestamps and added the doccomments to the overloads, because vscode didn't show them otherwiseab032f5Merge pull request #1 from heinrich26/main049b0b0Removed the old get_lyrics_with_timestamps method3198b1ffixed remaining issuescfd9a4aUpdate uploads.py0c2444cremoved variablecontextfrom the Mixin, as it's use was replaced byyt.as_mobile()a8a1f5ffix some formatting complaints by ruff and mypy📊 Changes
11 files changed (+236 additions, -48 deletions)
View changed files
📝
tests/mixins/test_browsing.py(+18 -2)📝
ytmusicapi/__init__.py(+1 -1)📝
ytmusicapi/mixins/_protocol.py(+7 -1)📝
ytmusicapi/mixins/_utils.py(+4 -1)📝
ytmusicapi/mixins/browsing.py(+83 -21)📝
ytmusicapi/mixins/library.py(+8 -6)📝
ytmusicapi/mixins/uploads.py(+8 -4)➕
ytmusicapi/models/__init__.py(+3 -0)➕
ytmusicapi/models/lyrics.py(+46 -0)📝
ytmusicapi/navigation.py(+10 -0)📝
ytmusicapi/ytmusic.py(+48 -12)📄 Description
Added
get_lyrics_with_timestampsto get lyrics with timestamps. The Method doesn't try to parse the response as normal lyrics, if no lyrics with timestamps are returned. (could be changed to do so tho, the format is the same)🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.