Skip to content
This repository has been archived by the owner on Jan 23, 2024. It is now read-only.

NaN for sync period #60

Closed
simon-weber opened this issue Apr 10, 2016 · 5 comments
Closed

NaN for sync period #60

simon-weber opened this issue Apr 10, 2016 · 5 comments
Labels

Comments

@simon-weber
Copy link
Owner

From #57, @PeteRepeat77 said:

I'm assuming my playlist wasn't updating because it had NaN in the sync period field? Is there a push value or does it need to be entered as an integer in minutes? I've changed mine to 60 minutes as I don't need the playlist updated dynamically really

I'm not sure how that happened. It's supposed to default to 1 minute. Was is set to NaN the first time you opened the settings page?

@PeteRepeat77
Copy link

Yes. I don't remember changing the sync setting ever. Pretty sure the extension is updated to the latest store version but can check when at home.

@simon-weber
Copy link
Owner Author

Huh, sorry about that. Something must have gone wrong adding the new setting.

@PeteRepeat77
Copy link

No problem. I wondered why the thumbs down songs (recently changed rating) were still in the playlist... Was because the playlist hadn't updated since I changed the conditions.

@simon-weber simon-weber self-assigned this Apr 10, 2016
@simon-weber
Copy link
Owner Author

I'm going to be completely honest: I have no idea how that happened, haha. This ought to fix it, though.

@simon-weber simon-weber removed their assignment Apr 11, 2016
@simon-weber
Copy link
Owner Author

Oh, and thanks for letting me know! I noticed a drop in syncs in GA, but I didn't think much of it since people tend to use the extension less on weekends.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants