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

wants to merge 3 commits into
base: master


Copy link

kauffecup commented Feb 3, 2016

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';

This comment has been minimized.

Copy link

Bontgoy Aug 22, 2017

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?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.