Documentation updates

[keneasson]

In the Documentation the Strategy function should be: FitbitOAuth2Strategy, and scope doesn't work if passed in the use.authentication, but needs to be in the arguments passed to the constructor.

var FitbitStrategy = require( 'passport-fitbit-oauth2' ).Strategy;

passport.use(new FitbitStrategy({
    clientID:     FITBIT_CLIENT_ID,
    clientSecret: FITBIT_CLIENT_SECRET,
    callbackURL: "http://yourdormain:3000/auth/fitbit/callback"
  },

As a Module, this works with the current code base:

var FitbitStrategy = require( 'passport-fitbit-oauth2' ).FitbitOAuth2Strategy;
var config = require('./fitbit_oauth2.js').settings;
module.exports = function(passport) {
    passport.use(new FitbitStrategy({
            clientID:     config.clientID,
            clientSecret: config.clientSecret,
            scope: config.scope,
            callbackURL: config.callbackURL,
        },
        function(accessToken, refreshToken, profile, done) {
            User.findOrCreate({ fitbitId: profile.id }, function (err, user) {
                return done(err, user);
            });
        }
    ));
}

[guumaster]

Doc upated after PR #1.

and scope doesn't work if passed in the use.authentication

Not sure what you mean by that.

[keneasson]

if i have in my route:
app.get('/auth/fitbit', passport.authenticate('fitbit',
{ scope: ['activity','heartrate','location','profile'] }
));
the FitBit api returns an error that scope wasn't set.
I needed to add scope as a property when I instantiate new FitbitStrategy. (as in my example above.)

[ghost]

Thanks for the great package. Please publish this doc change to npm, as npm's page for this package doesn't show the correct require information.