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

Update node examples to use spotify-web-api-node and es6 #7

Closed
wants to merge 3 commits into from

Conversation

kauffecup
Copy link

Updated authorization/app.js and client_credentials/app.js to use the npm module spotify-web-api-node and es6 syntax.

I was going through the "Beginner's Guide" which (which links to this repo), and found it would be beneficial to have updated examples using this library and a more modern syntax. It makes the examples more digestible and less complex (and also removed a bunch of lines! woo!)

The client code is untouched and is still fully functional with this updated node code

const cookieParser = require('cookie-parser');
const querystring = require('querystring');

const CLIENT_ID = '03ffe0cac0a0401aa6673c3cf6d02ced';
Copy link

Choose a reason for hiding this comment

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

Hi @kauffecup was browsing this repo as well and noticed your PR.

I like the updates but did you mean to leave your credentials in this pull request?

@kauffecup kauffecup closed this Mar 23, 2022
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