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.)

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.