Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Creation of top_music.py plugin #1241

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

SethSainz
Copy link

Add topmusic plugin using Spotify API for top 10 songs by genre. Has a pool of 28 different genres to choose from. Additionally, added spotipy to the requirements.txt file so that the plugin I created would be supported and recognized by Jarvis.

Using a command, such as "topmusic rock" the user will be provided by Jarvis with a list of the current top 10 songs of the provided genre on Spotify.

Created a topmusic plugin that utilizes the Spotify API to gather and display the current top 10 songs from a pool of 28 different possible genres. Added spotipy to the requirements.txt file so that the plugin would be supported and recognized by Jarvis.
Comment on lines +5 to +7
# Set up your credentials
client_id = '9432469329f94c449bc36b3273f7285a'
client_secret = '35d017ceebeb4eea83cd74ff5ee091ac'
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please don't put this in code. Please generate a new secret + make this secret invalid. We don't host credentials - features that need api key are allowed, but the user must provide an own api key.

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