[ACTIONS & SOURCES] Spotify new actions

[vellames]

Is there a specific app this action is for? Spotify

Please provide a link to the relevant API docs for the specific service / operation. https://developer.spotify.com/documentation/web-api/reference/

Actions

• [x] Create Playlist | Create a new playlist
• [x] Save Track | Save a track to "Your Music" library
• [x] Add Track to Playlist | Add a track to one of your playlists
• [x] Get Audio Features for Track | Get the audio features (duration, tempo, etc.) for a track
• [x] Find Track | Look up a track by title/artist or by entering a track ID
• [x] Artist's Top Tracks | Display artist's top 10 tracks
• [x] Get List of Category's Playlists in Genres & Moods | Retrieve playlists in each specific category in Genres & Moods
• [x] Get Playlist's List of Tracks | Retrieve a list of all tracks in a specified playlist
• [x] Remove Track | Delete the track with the given ID
• [x] Remove Track from Playlist | Delete the track with the given ID from a playlist

Addictional Improvements

• [x] Change app from CommonJS to ES Module
• [x] Change New Playlist Source from CommonJS to ES Module
• [x] Change New Saved Track Source from CommonJS to ES Module
• [x] Change New Track In Playlist Source from CommonJS to ES Module
• [x] Fixed a bug on New Track In Playlist Source. When a user added the same music to a distinct playlists, just one event was triggered

[vercel[bot]]

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployments, click below or on the icon next to each commit.

pipedream-docs-redirect-do-not-edit – ./docs

🔍 Inspect: https://vercel.com/pipedreamers/pipedream-docs-redirect-do-not-edit/8hvWdZmxzAfpbD5LUj5a1BWuwTWz
✅ Preview: https://pipedream-docs-redirect-do-not-edit-git-spo-f5404e-pipedreamers.vercel.app

pipedream-docs – ./docs

🔍 Inspect: https://vercel.com/pipedreamers/pipedream-docs/7sn5U6evmVkBZsdyoYGNS8U523Wi
✅ Preview: https://pipedream-docs-git-spotify-new-actions-pipedreamers.vercel.app

[dylburger]

• [x] Create components to address specific use cases whenever possible
• [x] Component keys should follow the format app_name_slug-slugified-component-name
• [x] Components should follow the standard directory structure
• [ ] Prefer Node.js client libraries to REST APIs
• [x] When making API requests, handle pagination to ensure all data / events are processed
• [ ] Use secret props to capture sensitive data
• [x] Props and methods should be defined in app files whenever possible
• [ ] Document methods with JS Docs
• [x] Use optional props whenever possible, and set a default value where you can
• [x] Use async options to accept user input wherever possible
• [ ] Create subscription webhooks sources (vs polling sources) whenever possible
• [ ] Always emit a summary with events
• [ ] Use a built-in deduping strategy wherever possible
• [ ] When building webhook sources, follow these best practices
• [ ] When building polling soures, follow these best practices

[vercel[bot]]

Deployment failed with the following error:

Creating the Deployment Timed Out.

[vellames]

@js07 Should I ping someone to merge this PR?

[dannyroosevelt]

@vellames-turing I can’t set the playlist_id for the “add_items_to_playlist” action. I think it’s trying to list IDs via async options, but I think it should just be a string, right?

Update: I think this is actually a bug within Pipedream! No action required right now, I'll update as soon as I know more.

[vellames]

@dannyroosevelt Yeah. Looks like a bug. I mentioned this before on Slack. If you click on save or deploy the changes are commited. This happens only if useQuery is true. Do you want me to open a new issue?

[dannyroosevelt]

@dannyroosevelt Yeah. Looks like a bug. I mentioned this before on Slack. If you click on save or deploy the changes are commited. This happens only if useQuery is true. Do you want me to open a new issue?

Actually I ran into an entirely separate bug that has since been fixed. I see the one you're referencing though, and Dylan has it on his radar.

[dannyroosevelt]

I can't get save-track to work. Either I get

{
  "error": {
    "status": 400,
    "message": "Malformed json payload"
  }
}

or no response returned (I believe I'm expecting a "Track saved" response).

@vellames-turing are you able to successfully save a track w/ this action?

[vellames]

Weird, is working properly for me. Can you send me your payload?

[dannyroosevelt]

@vellames-turing everything is working for me -- nice job! I just pushed a commit with a bunch of 1-line tweaks to most of the components for a new $export we're adding to the component API (not yet documented...I'm working on that).

$summary will be used to display a human readable success message in the soon-to-be-released new version of our workflow builder. Can you please review my changes and let me know if you have comments / edits, etc?

[vellames]

@vellames-turing everything is working for me -- nice job! I just pushed a commit with a bunch of 1-line tweaks to most of the components for a new $export we're adding to the component API (not yet documented...I'm working on that).

$summary will be used to display a human readable success message in the soon-to-be-released new version of our workflow builder. Can you please review my changes and let me know if you have comments / edits, etc?

The changes look good to me, there are some lint problems. I will fix, reset the version and check if I can improve the summary with the information that you have asked

[vellames]

@dannyroosevelt About the remove-items-from-playlist comment.

// we still show this even if no items were removed, since the API returns success and a "snapshot ID" -- is that weird?

We don't know if the music was removed using the return of this method. In my view, the message it's okay because even if there is nothing to be removed, the command was executed with success.

The snapshot looks okay for me. Its a Spotify control

[vellames]

@js07 @dannyroosevelt I implemented some changes. I tested all actions again. Looks like is everything working. Can you guys double-check it, please?

[vellames]

Hey @dannyroosevelt. I fixed the label/value problem in my last commit

[dannyroosevelt]

@vellames-turing FYI I made 2 of the tweaks Jacob recommended, I'll leave the rest to you

[vellames]

@js07 Requests applied

[vellames]

@dannyroosevelt I will take a look on it ASAP

[dannyroosevelt]

@vellames-turing it all looks good to me! merge away 😄