Skip to content

Use URI parameters as service params and remove bodyParser dependendency #77

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
daffl merged 1 commit into from
Jun 11, 2014
Merged

Use URI parameters as service params and remove bodyParser dependendency #77

daffl merged 1 commit into from
Jun 11, 2014

Conversation

@daffl
Copy link
Member

@daffl daffl commented Jun 11, 2014

This pull request adds req.params as service parameters so that routes like app.use('/:app/todos', todoService) will pass the app name as params.app. Also removes the body-parser hard dependency and makes you use your own. Closes #76 and closes #73.

@daffl daffl added this to the 1.0.0 milestone Jun 11, 2014
@daffl daffl mentioned this pull request Jun 11, 2014
Closed
daffl added a commit that referenced this pull request Jun 11, 2014
@daffl
Merge pull request #77 from feathersjs/bodyparser-params-update-76
7c0aec8 
Use URI parameters as service params and remove bodyParser dependendency
@daffl daffl merged commit 7c0aec8 into master Jun 11, 2014
@daffl daffl deleted the bodyparser-params-update-76 branch June 11, 2014 18:30
@daffl daffl mentioned this pull request Jun 18, 2014
Closed
daffl added a commit that referenced this pull request Aug 19, 2018
@daffl
Update all dependencies (#77)
ae4a328
daffl added a commit that referenced this pull request Aug 21, 2018
@daffl
Update all dependencies (#77)
242abae
daffl added a commit that referenced this pull request Aug 21, 2018
@daffl
Properly dispatch arrays that do not come from Feathers hook events (#77
71fe12d 
)

* Properly dispatch arrays that do not come from Feathers hook events

* Check if data is an array instead

* Remove unecessary code in tests
daffl pushed a commit that referenced this pull request Aug 21, 2018
@greenkeeper @daffl
Update @feathersjs/transport-commons to the latest version 🚀 (#77)
3be4274 
* fix(package): update @feathersjs/transport-commons to version 4.0.0

* Account for connection being passed
daffl added a commit that referenced this pull request Aug 22, 2018
@daffl
Properly dispatch arrays that do not come from Feathers hook events (#77
b242c07 
)

* Properly dispatch arrays that do not come from Feathers hook events

* Check if data is an array instead

* Remove unecessary code in tests
daffl pushed a commit that referenced this pull request Aug 22, 2018
@greenkeeper @daffl
Update @feathersjs/transport-commons to the latest version 🚀 (#77)
09aa3e3 
* fix(package): update @feathersjs/transport-commons to version 4.0.0

* Account for connection being passed
daffl added a commit that referenced this pull request Aug 25, 2018
@daffl
Merge pull request #77 from sod/auth-jslint-error
9e340cc 
jslint error in authentication
daffl added a commit that referenced this pull request Aug 29, 2018
@daffl
Updates for Feathers v3 (#77)
ab4fe83
daffl pushed a commit that referenced this pull request Aug 29, 2018
@ekryski @daffl
Merge pull request #77 from feathersjs/mobile-auth
811dccd 
Adding support for OAuth2 token based auth strategies. Closes #46.
daffl added a commit that referenced this pull request Aug 29, 2018
@daffl
Updates for Feathers v3 (#77)
7acf0a4
daffl pushed a commit that referenced this pull request Aug 29, 2018
@ekryski @daffl
Merge pull request #77 from feathersjs/mobile-auth
32e593e 
Adding support for OAuth2 token based auth strategies. Closes #46.
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

1.0.0

Development

Successfully merging this pull request may close these issues.

add route params to feathers params Remove bodyParser dependency

2 participants

@daffl @ekryski