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

Meetup.com Provider #146

Merged
merged 2 commits into from Sep 24, 2015
Merged

Meetup.com Provider #146

merged 2 commits into from Sep 24, 2015

Conversation

@Jamesford
Copy link
Contributor

Jamesford commented Sep 24, 2015

Using login with meetup.com for a project of mine, was implementing it using a custom bell provider, but thought others might be interested in this so I've decided to contribute back to bell. 😄

npm test results

...
meetup
  ✔ 71) authenticates with mock (12 ms)
...
79 tests complete
Test duration: 960 ms
No global variable leaks detected
Coverage: 100.00%
Linting results: No issues
@ldesplat

This comment has been minimized.

Copy link
Contributor

ldesplat commented Sep 24, 2015

Looks great! Thank you very much.

@ldesplat ldesplat added this to the 5.2.0 milestone Sep 24, 2015
@ldesplat ldesplat added the feature label Sep 24, 2015
ldesplat added a commit that referenced this pull request Sep 24, 2015
Meetup.com Provider
@ldesplat ldesplat merged commit 98a9c68 into hapijs:master Sep 24, 2015
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@ldesplat

This comment has been minimized.

Copy link
Contributor

ldesplat commented Sep 24, 2015

Published in 5.2.0! Thanks :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.