Skip to content

Merged #99 #100 #102 #103

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
Merged
merged 22 commits into from
Mar 28, 2016
Merged

Merged #99 #100 #102 #103

merged 22 commits into from
Mar 28, 2016

Conversation

guumaster
Copy link
Contributor

Merged #99 #100 #102 into one branch.

guumaster and others added 20 commits March 11, 2016 19:18
@guumaster
replaced config.json with config module.
74a9222
@alejandrogr
add public services to add and remove user realms
88ca174
@alejandrogr
internal services to add or remove user realms
169220a
@alejandrogr
add middleware for requiredBodyParams and findUserInPath
8882fcb
@alejandrogr
remove end parameter in redis.end (was added while testing)
093d24c
@alejandrogr
refactor routesUser test
4e4fa23
@alejandrogr
refactor to satisfy codeclimate
68e667f
@alejandrogr
remove duplicate endpoints
02aa69a
@alejandrogr
add test for dao.updateFieldWithMethod
feb6932
@alejandrogr
test for response_errors
a06b7c0
@alejandrogr
remove redis.end flush param (add while testing)
f8b4572
@alejandrogr
change configOptions signature in order to make it more readable
950a7c7
@alejandrogr
remove .only for routerUser test file
e1c2c15
@guumaster
updated mongodb driver to 2.x
ace5876
@guumaster
exclude tests paths from codeclimate.
e6487b6
@guumaster
extracted method findOne to reduce duplication
1dbf82a
@guumaster
removed pre-commit
cdff405
@guumaster
Merge branch 'feature/services_user_realms' into merged-branches
05bc11a 
# Conflicts:
#	src/managers/dao.js
@guumaster
merged feature/services_user_realms
75560be
@guumaster
Merge branch 'merged-branches' into all-pull-requests
cf3158c
This was referenced Mar 19, 2016
Closed
Closed
Closed
@guumaster guumaster changed the title Merged PRs Merged #99 #100 #102 Mar 19, 2016
@coveralls
Copy link

Coverage Status

Coverage increased (+1.3%) to 81.987% when pulling 1bc5874 on pr-99-100-102 into 585c4d5 on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+1.4%) to 82.008% when pulling 6c83c0c on pr-99-100-102 into 585c4d5 on master.

@luismesas luismesas merged commit 56ccbd0 into master Mar 28, 2016
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.
4 participants
@guumaster @coveralls @luismesas @alejandrogr