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.

Sign up for GitHub

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

Jump to bottom

Import changes from cfsghost wrt. GitHub API v3, add option to provide organization base URL, API #70

Open
wants to merge 50 commits into
base: master
Choose a base branch
from
Open

Import changes from cfsghost wrt. GitHub API v3, add option to provide organization base URL, API #70

wants to merge 50 commits into from

Conversation

benjspriggs
Copy link

I've imported all of the changes from master on cfsghost's repo that updated the Strategy to use GitHub's v3 of their API.

I've also:

  • Updated outdated dependencies (chai, mocha, proxyquire)
  • Added the option for the strategy to be configured with an alternative base URL and API URL, for Github Enterprise use cases
  • Added myself and cfsghost as contributors

cfsghost and others added 30 commits August 24, 2014 06:46
@cfsghost
Get email address via github API v3
ae891a4
@cfsghost
modified testcase for email API
a0f7e43
@cfsghost
* Disable node v0.4 on Travis CI
96abbc2 
* do not support node v0.4 anymore
@cfsghost
original author do not keep maintaining this module, so fork.
fb7a415
@cfsghost
Update Readme
07a5b10
@cfsghost
update example to use passport-github2
91be6e9
@cfsghost
bump version to 0.1.9
10d2ed5
@cfsghost
Update README.md
c85f3a8
@cfsghost
Update app.js
a08377e
@GregTurner
Update README.md
2aad1b6
@GregTurner
Merge pull request #1 from BCDevExchange/GregTurner-patch-1
5f3043f 
Update README.md
@cfsghost
Merge pull request #1 from BCDevExchange/master
9ee0c11 
Language corrections and NPM name
@colinterface
fix missing space in comment
14f87da
@cfsghost
Merge pull request #2 from ultralame/patch-1
be12a63 
fix missing space in comment
@hackreactor-students
update example to use express 4
a9953a8
@dougshamoo
Remove repeat word in app.js comment
0e801c2 
Remove repeat word 'function' in comment
@cfsghost
Merge pull request #3 from stevenlundy/master
fbc2385 
Update example to use express 4
@cfsghost
Merge pull request #4 from dougshamoo/patch-1
7773f34 
Remove repeat word in app.js comment
Fixed error message for emails endpoint error.
b54d090
@cfsghost
Merge pull request #5 from bkonetzny/emails-error-message
8602081 
Fixed error message for emails endpoint error.
@csilzen
README edit
3ed2d48 
Grammar fixes in intro blurb
@lipp
fix deprecated express middleware warnings
2e93928
@simison
Update Travis NodeJS versions
1ca2a12
@cfsghost
Merge pull request #6 from casandrawith1s/patch-1
2f4f222 
README edit
@cfsghost
Merge pull request #7 from lipp/fix-express-middleware-deprecated-usage
5c0c88d 
fix deprecated express middleware warnings
@cfsghost
Merge pull request #8 from simison/patch-2
d6557e2 
Update Travis NodeJS versions
@isayme
request _userEmailURL when has user:email permission
5d66773
@cfsghost
Merge pull request #9 from isayme/master
d2a2866 
request _userEmailURL when has user:email permission
@cfsghost
bump version to 0.1.10
d572f18
@cfsghost
Update README.md
a456666
skogsmaskin and others added 20 commits August 30, 2017 10:58
@skogsmaskin
Add avatar_url from Github to Passport's profile.photos
b274eb4
@cfsghost
Merge pull request #10 from skogsmaskin/avatar-url
5848696 
Add avatar_url from Github to Passport's profile.photos
@cfsghost
0.1.11
e36a279
@guoshencheng
add option for name
93e2777
@cfsghost
Merge pull request #11 from guoshencheng/master
87ac395 
add option for name
@benjspriggs
Resolve merge conflicts with passport-github
6e228c7
@benjspriggs
Update package.json with contributor, fork URL
3fa562e
@benjspriggs
Update package for new minor
fe27b3d
@benjspriggs
Update test to refer to avatar URL and assrt that photo is pulled int…
9a67da6 
…o profile, rather than from GraphQL
@benjspriggs
Update outdated dependencies
ee9b9ea
@benjspriggs
Update README
d183c9e
@benjspriggs
Remove errant symbols
5f060f3
@benjspriggs
Remove reference to deleted example directory
af00d58
@benjspriggs
Add options for providing a custom base and API URL
b186019
@benjspriggs
Update package.json
41c480e
@benjspriggs
Remove istanbul as dependency
436cd3e
@benjspriggs
Update travis config
bbca12e
@benjspriggs
Fix typo
711f9a8
@benjspriggs
Update node js versions - deprecate sub-v1.0 node
8c26ba9
@benjspriggs
Update node js versions - deprecate sub-v1.0 node
753106c
@coveralls
Copy link

coveralls commented Jan 23, 2018 •
edited
Loading

Coverage Status

Coverage remained the same at 100.0% when pulling 8c26ba9 on benjspriggs:master into 4db6a1b on jaredhanson:master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.