storytlr icon indicating copy to clipboard operation
storytlr copied to clipboard

Import Souncloud tracks uploaded date and tags

Open Simounet opened this issue 11 years ago • 5 comments

  • Handle created_at field from Soundcloud's tracks import so they can be at the right position in the timeline
  • Function to parse the tag_list field from the Soundcloud API
  • Add a created_at field to the soundcloud_data table
  • Var DATABASE_VERSION from index.php updated

Simounet avatar Jan 29 '14 18:01 Simounet

Thanks for your continued contributions :-)

I have an issue with this one: the soundcloud plugin currently only imports the user favorites (this can probably be easily extended to uploaded tracks as well). For favorites, I want the entry to be in my stream at the date/time when I favorited it, not when the track got uploaded (could have been years ago).

So, your fix would be the correct behavior for user uploaded tracks, but I don't like this behavior for favorites. What do you think? What use case did you have in mind?

eschnou avatar Jan 29 '14 20:01 eschnou

I agree. My next move will be to import our own tracks but I'll try to get the fav' date for favs before.

Simounet avatar Jan 29 '14 20:01 Simounet

Hewww, it doesn't seem to be possible atm: http://stackoverflow.com/a/20856588/2550448 . I've send a mail to the Soundcloud's API guys. I'll see if I can get a reply about this feature. In the mean time, can I split this PR with the tags feature, if it looks ok for you?

Simounet avatar Jan 29 '14 21:01 Simounet

Sure!

Aboute the date, I don't think it is too much an issue. My update cron job runs every five minutes. So, worst case is I'm off 5 minutes :-) The issue is when importing the feed the first time. So, one solution would be:

  • use the created timestamp on import (as long as we don't have anything better)
  • use the current timestamp when updating

eschnou avatar Jan 29 '14 21:01 eschnou

Hi Laurent, Time is passing by too fast... I made a patch and removed the created_at date handling. It's just for the tags' impot now.

Simounet avatar Nov 05 '14 17:11 Simounet