Skip to content

Conversation

@gAlleb
Copy link
Contributor

@gAlleb gAlleb commented Dec 14, 2024

Defining timestamp as time() in audioscrobbler.api.track.scrobble function makes time freeze and show the same time for all scrobbled tracks.

Снимок экрана 2024-12-14 в 20 20 26

All of the above tracks have been played through from start to finish. But all of them are scrobbled at the exact same time/time ago. It is the time when first track has been scrobbled since the start of LS session.

Using time() right inside params fixes it.

 params =
    [
      ("track", track),
      ("artist", artist),
      ("timestamp", string(time())),
...

#4250 (comment)

gAlleb and others added 2 commits December 15, 2024 00:24
Defining timestamp as `time()` in `audioscrobbler.api.track.scrobble` function makes it freeze and show the same time for all scrobbled tracks. Using `time()` right inside `params` fixes it.
@toots
Copy link
Member

toots commented Dec 15, 2024

Thanks!

@toots toots enabled auto-merge December 15, 2024 08:52
@toots toots added this pull request to the merge queue Dec 15, 2024
Merged via the queue into savonet:main with commit 95fb55b Dec 15, 2024
30 checks passed
toots added a commit that referenced this pull request Jan 3, 2025
Co-authored-by: Romain Beauxis <toots@rastageeks.org>
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